Skip to content

move coverage to test_nightly#304

Draft
mikaelarguedas wants to merge 1 commit intorollingfrom
move_coverage_to_test_nightly
Draft

move coverage to test_nightly#304
mikaelarguedas wants to merge 1 commit intorollingfrom
move_coverage_to_test_nightly

Conversation

@mikaelarguedas
Copy link
Member

as coverage doesnt depend on dependencies and test_latest takes forever

@mikaelarguedas mikaelarguedas force-pushed the move_coverage_to_test_nightly branch from c24d0cd to 3b1c61d Compare May 8, 2024 15:56
@codecov
Copy link

codecov bot commented May 8, 2024

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 76.29%. Comparing base (702db7a) to head (47ed9a5).
⚠️ Report is 1 commits behind head on rolling.

Additional details and impacted files
@@             Coverage Diff              @@
##           rolling     #304       +/-   ##
============================================
- Coverage    88.38%   76.29%   -12.10%     
============================================
  Files           24       24               
  Lines          620      620               
  Branches        66       66               
============================================
- Hits           548      473       -75     
- Misses          52      134       +82     
+ Partials        20       13        -7     
Flag Coverage Δ
unittests 76.29% <ø> (-12.10%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@mikaelarguedas
Copy link
Member Author

for some reason I have the same odd behavior locally:

  • if installed from packages or source: coverage sros2 coverage 78%
  • if using the nightly archive and building

While test_generate policy passes it doesnt appear in coverage results.

Stalling this for now

@mikaelarguedas mikaelarguedas force-pushed the move_coverage_to_test_nightly branch 2 times, most recently from 1eeb328 to ab20661 Compare May 8, 2024 17:27
Signed-off-by: Mikael Arguedas <mikael.arguedas@gmail.com>
@mikaelarguedas mikaelarguedas force-pushed the move_coverage_to_test_nightly branch from ab20661 to 47ed9a5 Compare July 30, 2025 03:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant