Skip to content

Javascript CI/CD integration#653

Merged
K20shores merged 5 commits intomainfrom
CICD
Oct 15, 2025
Merged

Javascript CI/CD integration#653
K20shores merged 5 commits intomainfrom
CICD

Conversation

@sicou2
Copy link
Copy Markdown
Collaborator

@sicou2 sicou2 commented Oct 15, 2025

closes #620

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Oct 15, 2025

📄 Documentation for this branch is available at: https://ncar.github.io/musica/branch/CICD/

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Oct 15, 2025 •
edited
Loading

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 78.24%. Comparing base (892f4b0) to head (3d4070f).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files 
@@           Coverage Diff           @@
##             main     #653   +/-   ##
=======================================
  Coverage   78.24%   78.24%           
=======================================
  Files          54       54           
  Lines        6857     6857           
=======================================
  Hits         5365     5365           
  Misses       1492     1492

☔ 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 requested changes Oct 15, 2025
View reviewed changes
Copy link
Copy Markdown
Collaborator

@K20shores K20shores left a comment

Choose a reason for hiding this comment

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

Please add a badge to show these tests on the readme. The readme has examples of this for some of our tests

@sicou2
Copy link
Copy Markdown
Collaborator Author

sicou2 commented Oct 15, 2025

Please add a badge to show these tests on the readme. The readme has examples of this for some of our tests

This is done.

K20shores
K20shores requested changes Oct 15, 2025
View reviewed changes
Copy link
Copy Markdown
Collaborator

@K20shores K20shores left a comment

Choose a reason for hiding this comment

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

One last thing, but well done!

@sicou2
Removes sync-version test step from workflow
3d4070f 
Removed the test step for syncing version in the CI workflow.
@K20shores K20shores merged commit 45b25d1 into main Oct 15, 2025
72 of 74 checks passed
@K20shores K20shores deleted the CICD branch October 15, 2025 21:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@K20shores K20shores K20shores approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Add javascript tests, CI/CD

3 participants

@sicou2 @codecov-commenter @K20shores