Skip to content

Support reinstall in check command#1001

Open
krisl wants to merge 17 commits intoros-infrastructure:masterfrom
krisl:support-reinstall-in-check-command
Open

Support reinstall in check command#1001
krisl wants to merge 17 commits intoros-infrastructure:masterfrom
krisl:support-reinstall-in-check-command

Conversation

@krisl
Copy link
Contributor

@krisl krisl commented Apr 27, 2025

Adds --reinstall support also for the check command, so we can get a full list of deps.
Useful for things like CI and packaging where you want to spit out all dependencies to do something with them later on.

Additionally adds some consistency in output between commands.

@krisl krisl force-pushed the support-reinstall-in-check-command branch 2 times, most recently from 0f7160d to a3f989d Compare May 18, 2025 08:26
@krisl krisl force-pushed the support-reinstall-in-check-command branch from a3f989d to b7dec8e Compare August 21, 2025 20:22
@codecov
Copy link

codecov bot commented Aug 21, 2025

Codecov Report

❌ Patch coverage is 60.00000% with 14 lines in your changes missing coverage. Please review.
✅ Project coverage is 72.43%. Comparing base (ad3a610) to head (b7dec8e).
⚠️ Report is 3 commits behind head on master.

Files with missing lines Patch % Lines
src/rosdep2/main.py 46.15% 11 Missing and 3 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1001      +/-   ##
==========================================
+ Coverage   71.76%   72.43%   +0.67%     
==========================================
  Files          44       44              
  Lines        3488     3483       -5     
  Branches      686      684       -2     
==========================================
+ Hits         2503     2523      +20     
+ Misses        808      785      -23     
+ Partials      177      175       -2     

☔ 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.

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