Skip to content

Add check command for potential pack updates#2459

Merged
jthuangarm merged 15 commits intoOpen-CMSIS-Pack:mainfrom
jthuangarm:check-command
Apr 28, 2026
Merged

Add check command for potential pack updates#2459
jthuangarm merged 15 commits intoOpen-CMSIS-Pack:mainfrom
jthuangarm:check-command

Conversation

@jthuangarm
Copy link
Copy Markdown
Contributor

@jthuangarm jthuangarm commented Apr 27, 2026

Fixes

Changes

  • Added the check command to the CLI, allowing users to check for updates to packs used in their projects. This includes parsing the command, handling input validation, and integrating the new functionality into the main command processor.

Here is a real output from an existing project:

>csolution check *.csolution.yml -v
ARM::CMSIS@6.3.0 (up-to-date)
ARM::CMSIS-Compiler@2.0.0 -> 2.2.0
  Release notes for v2.2.0:
      Changes since CMSIS-Compiler 2.1.0:
      - Add components that enable stdio retarget to UART
      - Update example project
      AC6:
      - correct _sys_read return value when rt_fs_read returns with error
      - correct _mutex_initialize to check whether RTOS2 kernel is initialized
      IAR:
      - add support for multithreading
      - add support for file redirection
  Release notes for v2.1.0:
      - AC6: correct _sys_open handling for OPEN_A, file is created if non-existent (#50)
      - GCC: add missing forward declarations for retargeted functions (#51, #53)
      - Add Cortex-M52 support
      - GCC: Disable heap limit check in _sbrk_r implementation (#54)
...
AlifSemiconductor::Ensemble@2.1.0 -> 2.2.0-dev
  Release notes for v2.2.0-dev:
      Active development...
Keil::MDK-Middleware@8.2.0 (up-to-date)
PyTorch::ExecuTorch@1.1.0-rc1-build.12 (up-to-date)
  Local path: ../../packs/PyTorch.ExecuTorch.1.1.0-rc1-build.12

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

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 27, 2026

Test Results

  7 files   -    51   53 suites   - 123   5m 42s ⏱️ - 15m 40s
185 tests  -   725  168 ✅  -   725  17 💤 ±0  0 ❌ ±0 
692 runs   - 2 175  624 ✅  - 2 175  68 💤 ±0  0 ❌ ±0 

Results for commit 8ac1cf8. ± Comparison against base commit 79b66b5.

This pull request removes 725 tests.
CodeGenerator ‑ Check
CrossPlatformUnitTests ‑ CanExecute
CrossPlatformUnitTests ‑ ExecCommand
CrossPlatformUnitTests ‑ GetEnv_Empty
CrossPlatformUnitTests ‑ GetEnv_Existing
CrossPlatformUnitTests ‑ GetEnv_NoExising
CrossPlatformUnitTests ‑ GetExecutablePath
CrossPlatformUnitTests ‑ GetLongPathRegStatus
CrossPlatformUnitTests ‑ GetPackRootDir_Default
CrossPlatformUnitTests ‑ GetPackRootDir_NoEnvSet
…

♻️ This comment has been updated with latest results.

@jthuangarm jthuangarm marked this pull request as ready for review April 27, 2026 15:26
@jthuangarm jthuangarm requested a review from brondani April 28, 2026 07:12
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 28, 2026

Codecov Report

❌ Patch coverage is 63.80952% with 38 lines in your changes missing coverage. Please review.
✅ Project coverage is 64.93%. Comparing base (79b66b5) to head (afcf91d).

Files with missing lines Patch % Lines
tools/projmgr/src/ProjMgrWorker.cpp 62.92% 17 Missing and 16 partials ⚠️
tools/projmgr/src/ProjMgr.cpp 68.75% 2 Missing and 3 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2459      +/-   ##
==========================================
- Coverage   64.93%   64.93%   -0.01%     
==========================================
  Files         145      145              
  Lines       26209    26314     +105     
  Branches    15828    15899      +71     
==========================================
+ Hits        17020    17088      +68     
- Misses       7042     7061      +19     
- Partials     2147     2165      +18     
Flag Coverage Δ
buildmgr-cov 79.87% <ø> (ø)
packchk-cov 69.75% <ø> (ø)
projmgr-cov 87.93% <63.80%> (-0.29%) ⬇️

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/include/ProjMgrWorker.h 83.33% <ø> (ø)
tools/projmgr/src/ProjMgr.cpp 86.79% <68.75%> (-0.28%) ⬇️
tools/projmgr/src/ProjMgrWorker.cpp 87.30% <62.92%> (-0.57%) ⬇️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@jthuangarm jthuangarm requested a review from brondani April 28, 2026 12:08
Copy link
Copy Markdown
Collaborator

@brondani brondani left a comment

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 564399d into Open-CMSIS-Pack:main Apr 28, 2026
85 of 86 checks passed
@jthuangarm jthuangarm deleted the check-command branch April 28, 2026 16:01
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 28, 2026

Codecov Report

❌ Patch coverage is 63.04348% with 34 lines in your changes missing coverage. Please review.
✅ Project coverage is 64.93%. Comparing base (79b66b5) to head (8ac1cf8).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
tools/projmgr/src/ProjMgrWorker.cpp 61.84% 15 Missing and 14 partials ⚠️
tools/projmgr/src/ProjMgr.cpp 68.75% 2 Missing and 3 partials ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2459      +/-   ##
==========================================
- Coverage   64.93%   64.93%   -0.01%     
==========================================
  Files         145      145              
  Lines       26209    26301      +92     
  Branches    15828    15888      +60     
==========================================
+ Hits        17020    17078      +58     
- Misses       7042     7059      +17     
- Partials     2147     2164      +17     
Flag Coverage Δ
buildmgr-cov 79.87% <ø> (ø)
packchk-cov 69.75% <ø> (ø)
projmgr-cov 87.95% <63.04%> (-0.27%) ⬇️

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/include/ProjMgrWorker.h 83.33% <ø> (ø)
tools/projmgr/src/ProjMgr.cpp 86.79% <68.75%> (-0.28%) ⬇️
tools/projmgr/src/ProjMgrWorker.cpp 87.34% <61.84%> (-0.53%) ⬇️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Copy Markdown
Contributor

@soumeh01 soumeh01 left a comment

Choose a reason for hiding this comment

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

@jthuangarm Thanks for making these contributions.
Just a nit-pick, please make sure to update the usage documentation
https://github.com/Open-CMSIS-Pack/cmsis-toolbox/blob/main/docs/build-tools.md#csolution-invocation

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.

3 participants