-
Notifications
You must be signed in to change notification settings - Fork 374
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Run specs on GitHub Actions #3735
base: master
Are you sure you want to change the base?
Conversation
Draft PR, please |
a04442d
to
b26071f
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #3735 +/- ##
==========================================
- Coverage 98.10% 98.10% -0.01%
==========================================
Files 1225 1225
Lines 72866 72847 -19
Branches 3493 3494 +1
==========================================
- Hits 71485 71466 -19
Misses 1381 1381 ☔ View full report in Codecov by Sentry. |
fatal: detected dubious ownership in repository at '/__w/dd-trace-rb/dd-trace-rb' To add an exception for this directory, call: git config --global --add safe.directory /__w/dd-trace-rb/dd-trace-rb This creeps up e.g when container user info doesn't match host info (e.g via a host volume mount).
This works around GHA issues with a lot of parallel jobs across workflows.
4d67104
to
b40032a
Compare
BenchmarksBenchmark execution time: 2024-09-10 13:34:11 Comparing candidate commit 4655ea7 in PR branch Found 0 performance improvements and 0 performance regressions! Performance is the same for 23 metrics, 2 unstable metrics. |
28c2617
to
68db818
Compare
Hey @lloeki is this still relevant? |
What does this PR do?
Guess.
Motivation:
Hold my beer.
Additional Notes:
Trying stuff. May or may not work. But it allows me to test https://github.com/DataDog/images-rb without breaking CircleCI and something may or may not come out of it.
How to test the change?
Wait for it.