Skip to content

Bump GitHub Actions to latest pinned versions#87

Merged
QilongTang merged 3 commits into
masterfrom
copilot/fix-build-workflow-code-coverage
Apr 27, 2026
Merged

Bump GitHub Actions to latest pinned versions#87
QilongTang merged 3 commits into
masterfrom
copilot/fix-build-workflow-code-coverage

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 27, 2026

Floating @v6 tags on first-party actions and a stale SHA on zgosalvez/github-actions-report-lcov left workflows running outdated code.

Changes

  • actions/checkout@v6de0fac2e (v6.0.2) in build.yml and npm-publish.yml
  • actions/setup-node@v648b55a01 (v6.4.0) in build.yml and npm-publish.yml
  • zgosalvez/github-actions-report-lcov5aba2ff (v7.0.9) → 7d72c57c (v7.0.10) in build.yml (dep bumps: fast-xml-parser, @actions/github, @actions/glob)
  • hrishikesh-kadam/setup-lcov was already current — no change

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates CI/CD workflows to pin GitHub Actions to specific commit SHAs (instead of floating @v6 tags) and refreshes the pinned SHA for the LCOV report action, ensuring workflows execute the intended, up-to-date action code.

Changes:

  • Pin actions/checkout and actions/setup-node to specific SHAs in both build and publish workflows.
  • Bump zgosalvez/github-actions-report-lcov to the latest pinned SHA and explicitly provide github-token in the build workflow.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
.github/workflows/npm-publish.yml Pins actions/checkout and actions/setup-node to commit SHAs for deterministic release builds/publishing.
.github/workflows/build.yml Pins actions/checkout/setup-node, bumps github-actions-report-lcov, and passes GITHUB_TOKEN for coverage reporting.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@dynamo-ci-user
Copy link
Copy Markdown

LCOV of commit eeec4eb during Build #420

Summary coverage rate:
  lines......: 85.3% (483 of 566 lines)
  functions..: 80.0% (108 of 135 functions)
  branches...: no data found

Files changed coverage rate: n/a

@avidit avidit marked this pull request as ready for review April 27, 2026 14:59
@avidit
Copy link
Copy Markdown
Contributor

avidit commented Apr 27, 2026

Also added github-token with appropriate persmission to publish coverage report as PR comment. example above #87 (comment)

@avidit avidit requested a review from QilongTang April 27, 2026 15:02
@QilongTang QilongTang merged commit 0c7bb72 into master Apr 27, 2026
11 checks passed
@QilongTang QilongTang deleted the copilot/fix-build-workflow-code-coverage branch April 27, 2026 15:30
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.

5 participants