Skip to content

Refactor version reporting for consistent API across all languages#696

Merged
K20shores merged 5 commits intomainfrom
copilot/refactor-version-reporting
Dec 9, 2025
Merged

Refactor version reporting for consistent API across all languages#696
K20shores merged 5 commits intomainfrom
copilot/refactor-version-reporting

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Dec 3, 2025

  • Refactors python so that all package versions are retrieved with __version__ in their init.pys
  • The musica cli now reports the version for each package
  • Adds the micm versions to javascript
  • Removes the component version stuff because it wasn't used anywhere
  • adds tuvx, musica versions to the fortran interface with tests

Fixes #695

Copilot AI changed the title [WIP] Refactor version reporting for musica and its packages Refactor version reporting for consistent API across all languages Dec 3, 2025
Copilot AI requested a review from K20shores December 3, 2025 21:55
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Dec 3, 2025

Codecov Report

❌ Patch coverage is 80.00000% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 81.47%. Comparing base (4713fe6) to head (77c0743).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
python/musica/main.py 0.00% 3 Missing ⚠️
src/tuvx/tuvx_c_interface.cpp 0.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #696      +/-   ##
==========================================
- Coverage   81.52%   81.47%   -0.06%     
==========================================
  Files         114      112       -2     
  Lines        9921     9894      -27     
==========================================
- Hits         8088     8061      -27     
  Misses       1833     1833              
Flag Coverage Δ
cpp_fortran 79.60% <81.81%> (-0.12%) ⬇️
javascript 92.12% <ø> (ø)
python 82.66% <78.57%> (+0.09%) ⬆️

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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@K20shores K20shores force-pushed the copilot/refactor-version-reporting branch from 0aa3ad1 to 48ab4f7 Compare December 4, 2025 00:02
- placed all package componenet versions in __init__.py for that
  component
- added all versions to the --version flag of the cli tool
- removing old header file
- updating test:unit to run all tests
- correcting npm test path
- removing file added by copilot
@K20shores K20shores force-pushed the copilot/refactor-version-reporting branch from dc9a5fa to 77c0743 Compare December 4, 2025 22:05
@K20shores K20shores marked this pull request as ready for review December 4, 2025 23:59
@K20shores K20shores merged commit 4e533bc into main Dec 9, 2025
81 checks passed
@K20shores K20shores deleted the copilot/refactor-version-reporting branch December 9, 2025 19:43
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.

Refactor version reporting for musica and all of it's packages

4 participants