remove junit upload action - #8148
Conversation
Overall package sizeSelf size: 5.67 MB Dependency sizes| name | version | self size | total size | |------|---------|-----------|------------| | import-in-the-middle | 3.0.1 | 82.56 kB | 817.39 kB | | dc-polyfill | 0.1.10 | 26.73 kB | 26.73 kB |🤖 This report was automatically generated by heaviest-objects-in-the-universe |
🎉 All green!❄️ No new flaky tests detected 🎯 Code Coverage (details) 🔗 Commit SHA: f74dac7 | Docs | Datadog PR Page | Give us feedback! |
BenchmarksBenchmark execution time: 2026-04-29 14:34:51 Comparing candidate commit f74dac7 in PR branch Found 0 performance improvements and 0 performance regressions! Performance is the same for 1336 metrics, 108 unstable metrics. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 2a91c8d9d6
ℹ️ About Codex in GitHub
Codex has been enabled to automatically review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
When you sign up for Codex through ChatGPT, Codex can also answer questions or update the PR, like "@codex address that feedback".
juan-fernandez
left a comment
There was a problem hiding this comment.
potentially missing test data
juan-fernandez
left a comment
There was a problem hiding this comment.
the number of reported tests looks OK now
ab2d959 to
f74dac7
Compare
What does this PR do?
Remove JUnit upload action.
Motivation
We are having a lot of issues with installs from GitHub Release right now, which the JUnit upload action uses. For now it's simpler to just switch back to the npm approach, which is actually simple enough that we can just have it in dd-trace, no need for a separate action. If we make further improvements or feel the need in the future to upstream our changes we can do that, but for now it's simpler to work on the immediate issue if this code is local and not in 2 transitive actions.
Additional Notes