Skip to content

[projmgr] Use check pack-updates instead of check#2464

Merged
jthuangarm merged 1 commit into
Open-CMSIS-Pack:mainfrom
jthuangarm:checkPackUpdates
Apr 29, 2026
Merged

[projmgr] Use check pack-updates instead of check#2464
jthuangarm merged 1 commit into
Open-CMSIS-Pack:mainfrom
jthuangarm:checkPackUpdates

Conversation

@jthuangarm

@jthuangarm jthuangarm commented Apr 29, 2026

Copy link
Copy Markdown
Contributor

Fixes

Changes

  • Changed the check command to require a subcommand, currently supporting check pack-updates, and updated command-line parsing and help text to reflect this new structure.

Checklist

  • 🤖 This change is covered by unit tests (if applicable).
  • 🤹 Manual testing has been performed (if necessary).
  • 🛡️ Security impacts have been considered (if relevant).
  • 📖 Documentation updates are complete (if required).
  • 🧠 Third-party dependencies and TPIP updated (if required).

@jthuangarm jthuangarm changed the title [projmgr] Use check updates instead of check [projmgr] Use check pack-updates instead of check Apr 29, 2026
@jthuangarm jthuangarm marked this pull request as ready for review April 29, 2026 12:11
@jthuangarm jthuangarm requested a review from brondani April 29, 2026 12:11
@github-actions

Copy link
Copy Markdown

Test Results

    3 files     21 suites   16m 3s ⏱️
  461 tests   461 ✅ 0 💤 0 ❌
1 383 runs  1 383 ✅ 0 💤 0 ❌

Results for commit 62ee477.

@brondani brondani left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jthuangarm jthuangarm merged commit d511ad6 into Open-CMSIS-Pack:main Apr 29, 2026
33 checks passed
@jthuangarm jthuangarm deleted the checkPackUpdates branch April 29, 2026 12:41
@codecov

codecov Bot commented Apr 29, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 53.84615% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 64.93%. Comparing base (281a898) to head (62ee477).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
tools/projmgr/src/ProjMgr.cpp 53.84% 4 Missing and 2 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2464      +/-   ##
==========================================
- Coverage   64.94%   64.93%   -0.02%     
==========================================
  Files         145      145              
  Lines       26311    26321      +10     
  Branches    15896    15902       +6     
==========================================
+ Hits        17088    17091       +3     
- Misses       7059     7063       +4     
- Partials     2164     2167       +3     
Flag Coverage Δ
projmgr-cov 87.90% <53.84%> (-0.07%) ⬇️

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

Files with missing lines Coverage Δ
tools/projmgr/include/ProjMgr.h 100.00% <ø> (ø)
tools/projmgr/src/ProjMgr.cpp 86.14% <53.84%> (-0.65%) ⬇️

... and 1 file with indirect coverage changes

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

2 participants