-
-
Notifications
You must be signed in to change notification settings - Fork 6.5k
chore(ci): bump scorecard #8500
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: Aviv Keller <me@aviv.sh>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
👋 Codeowner Review RequestThe following codeowners have been identified for the changed files: Team reviewers: @nodejs/web-infra Please review the changes when you have a chance. Thank you! 🙏 |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #8500 +/- ##
=======================================
Coverage 73.73% 73.73%
=======================================
Files 108 108
Lines 9210 9210
Branches 312 312
=======================================
Hits 6791 6791
Misses 2417 2417
Partials 2 2 ☔ View full report in Codecov by Sentry. |
Signed-off-by: Aviv Keller <me@aviv.sh>
There was a problem hiding this 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 the scorecard workflow reference to use a newer commit from the nodejs/web-team repository. The change is blocked by an upstream PR (nodejs/web-team#85) that presumably introduces the changes in the referenced commit.
- Updates the scorecard workflow to reference commit
64d15ba684371b26784aa58680ea3f31120213bafrom nodejs/web-team
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
📦 Build Size ComparisonSummary
Changes➕ Added Assets (1)
➖ Removed Assets (1)
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We should probably also switch to permissions: {} at the top-level here too?
Blocked by nodejs/web-team#85