Skip to content

Conversation

@martincostello
Copy link
Contributor

Resolves#1858.

CopilotAI review requested due to automatic review settings August 16, 2025 09:30
Copy link

CopilotAI 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 pins the actions/github-script action to a specific Git SHA for improved security by preventing potential supply chain attacks through version pinning.

  • Replaces version tag reference with SHA-based pinning for actions/github-script

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@joseph-sentryjoseph-sentry merged commit 39a2af1 into codecov:mainAug 17, 2025
11 checks passed
@codecov
Copy link

codecovbot commented Aug 17, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 0.00%. Comparing base (2db07e3) to head (bba17ef).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@ Coverage Diff @@## main #1859 +/- ## ============================ ============================ 

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

nodejs-github-bot pushed a commit to nodejs/node that referenced this pull request Aug 18, 2025
Refs: codecov/codecov-action#1859 Refs: codecov/codecov-action#1858 PR-URL: #59512 Reviewed-By: Filip Skokan <[email protected]> Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Zeyu "Alex" Yang <[email protected]> Reviewed-By: Moshe Atlow <[email protected]> Reviewed-By: Daeyeon Jeong <[email protected]> Reviewed-By: Pietro Marchini <[email protected]>
panva pushed a commit to panva/node that referenced this pull request Aug 18, 2025
Refs: codecov/codecov-action#1859 Refs: codecov/codecov-action#1858 PR-URL: nodejs#59512 Reviewed-By: Filip Skokan <[email protected]> Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Zeyu "Alex" Yang <[email protected]> Reviewed-By: Moshe Atlow <[email protected]> Reviewed-By: Daeyeon Jeong <[email protected]> Reviewed-By: Pietro Marchini <[email protected]>
@viceice
Copy link

Please publish a new release with this fix

@martincostellomartincostello deleted the patch-1 branch August 18, 2025 12:16
marco-ippolito pushed a commit to nodejs/node that referenced this pull request Aug 20, 2025
Refs: codecov/codecov-action#1859 Refs: codecov/codecov-action#1858 PR-URL: #59512 Reviewed-By: Filip Skokan <[email protected]> Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Zeyu "Alex" Yang <[email protected]> Reviewed-By: Moshe Atlow <[email protected]> Reviewed-By: Daeyeon Jeong <[email protected]> Reviewed-By: Pietro Marchini <[email protected]>
marco-ippolito pushed a commit to nodejs/node that referenced this pull request Aug 20, 2025
Refs: codecov/codecov-action#1859 Refs: codecov/codecov-action#1858 PR-URL: #59512 Reviewed-By: Filip Skokan <[email protected]> Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Zeyu "Alex" Yang <[email protected]> Reviewed-By: Moshe Atlow <[email protected]> Reviewed-By: Daeyeon Jeong <[email protected]> Reviewed-By: Pietro Marchini <[email protected]>
aduh95 pushed a commit to nodejs/node that referenced this pull request Aug 21, 2025
Refs: codecov/codecov-action#1859 Refs: codecov/codecov-action#1858 PR-URL: #59512 Reviewed-By: Filip Skokan <[email protected]> Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Zeyu "Alex" Yang <[email protected]> Reviewed-By: Moshe Atlow <[email protected]> Reviewed-By: Daeyeon Jeong <[email protected]> Reviewed-By: Pietro Marchini <[email protected]>
targos pushed a commit to nodejs/node that referenced this pull request Aug 21, 2025
Refs: codecov/codecov-action#1859 Refs: codecov/codecov-action#1858 PR-URL: #59512 Reviewed-By: Filip Skokan <[email protected]> Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Zeyu "Alex" Yang <[email protected]> Reviewed-By: Moshe Atlow <[email protected]> Reviewed-By: Daeyeon Jeong <[email protected]> Reviewed-By: Pietro Marchini <[email protected]>
marco-ippolito pushed a commit to nodejs/node that referenced this pull request Aug 23, 2025
Refs: codecov/codecov-action#1859 Refs: codecov/codecov-action#1858 PR-URL: #59512 Reviewed-By: Filip Skokan <[email protected]> Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Zeyu "Alex" Yang <[email protected]> Reviewed-By: Moshe Atlow <[email protected]> Reviewed-By: Daeyeon Jeong <[email protected]> Reviewed-By: Pietro Marchini <[email protected]>
marco-ippolito pushed a commit to nodejs/node that referenced this pull request Aug 25, 2025
Refs: codecov/codecov-action#1859 Refs: codecov/codecov-action#1858 PR-URL: #59512 Reviewed-By: Filip Skokan <[email protected]> Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Zeyu "Alex" Yang <[email protected]> Reviewed-By: Moshe Atlow <[email protected]> Reviewed-By: Daeyeon Jeong <[email protected]> Reviewed-By: Pietro Marchini <[email protected]>
marco-ippolito pushed a commit to nodejs/node that referenced this pull request Aug 25, 2025
Refs: codecov/codecov-action#1859 Refs: codecov/codecov-action#1858 PR-URL: #59512 Reviewed-By: Filip Skokan <[email protected]> Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Zeyu "Alex" Yang <[email protected]> Reviewed-By: Moshe Atlow <[email protected]> Reviewed-By: Daeyeon Jeong <[email protected]> Reviewed-By: Pietro Marchini <[email protected]>
marco-ippolito pushed a commit to nodejs/node that referenced this pull request Aug 25, 2025
Refs: codecov/codecov-action#1859 Refs: codecov/codecov-action#1858 PR-URL: #59512 Reviewed-By: Filip Skokan <[email protected]> Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Zeyu "Alex" Yang <[email protected]> Reviewed-By: Moshe Atlow <[email protected]> Reviewed-By: Daeyeon Jeong <[email protected]> Reviewed-By: Pietro Marchini <[email protected]>
marco-ippolito pushed a commit to nodejs/node that referenced this pull request Aug 25, 2025
Refs: codecov/codecov-action#1859 Refs: codecov/codecov-action#1858 PR-URL: #59512 Reviewed-By: Filip Skokan <[email protected]> Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Zeyu "Alex" Yang <[email protected]> Reviewed-By: Moshe Atlow <[email protected]> Reviewed-By: Daeyeon Jeong <[email protected]> Reviewed-By: Pietro Marchini <[email protected]>
aduh95 pushed a commit to nodejs/node that referenced this pull request Aug 25, 2025
Refs: codecov/codecov-action#1859 Refs: codecov/codecov-action#1858 PR-URL: #59512 Reviewed-By: Filip Skokan <[email protected]> Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Zeyu "Alex" Yang <[email protected]> Reviewed-By: Moshe Atlow <[email protected]> Reviewed-By: Daeyeon Jeong <[email protected]> Reviewed-By: Pietro Marchini <[email protected]>
aduh95 pushed a commit to nodejs/node that referenced this pull request Aug 26, 2025
Refs: codecov/codecov-action#1859 Refs: codecov/codecov-action#1858 PR-URL: #59512 Reviewed-By: Filip Skokan <[email protected]> Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Zeyu "Alex" Yang <[email protected]> Reviewed-By: Moshe Atlow <[email protected]> Reviewed-By: Daeyeon Jeong <[email protected]> Reviewed-By: Pietro Marchini <[email protected]>
marco-ippolito pushed a commit to nodejs/node that referenced this pull request Aug 26, 2025
Refs: codecov/codecov-action#1859 Refs: codecov/codecov-action#1858 PR-URL: #59512 Reviewed-By: Filip Skokan <[email protected]> Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Zeyu "Alex" Yang <[email protected]> Reviewed-By: Moshe Atlow <[email protected]> Reviewed-By: Daeyeon Jeong <[email protected]> Reviewed-By: Pietro Marchini <[email protected]>
marco-ippolito pushed a commit to nodejs/node that referenced this pull request Aug 27, 2025
Refs: codecov/codecov-action#1859 Refs: codecov/codecov-action#1858 PR-URL: #59512 Reviewed-By: Filip Skokan <[email protected]> Reviewed-By: Michaël Zasso <[email protected]> Reviewed-By: Zeyu "Alex" Yang <[email protected]> Reviewed-By: Moshe Atlow <[email protected]> Reviewed-By: Daeyeon Jeong <[email protected]> Reviewed-By: Pietro Marchini <[email protected]>
Sign up for freeto 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.

[BUG] Failure due to actions/github-script@v7 when pinned SHA is required

3 participants

@martincostello@viceice@joseph-sentry