-
-
Notifications
You must be signed in to change notification settings - Fork 6.5k
Blog: add update to Security CI incident #7658
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
Conversation
Co-Authored-By: Antoine du Hamel <[email protected]> Co-Authored-By: Richard Lau <[email protected]> Co-Authored-By: Matteo Collina <[email protected]>
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
apps/site/pages/en/blog/vulnerability/march-2025-ci-incident.md
Outdated
Show resolved
Hide resolved
apps/site/pages/en/blog/vulnerability/march-2025-ci-incident.md
Outdated
Show resolved
Hide resolved
apps/site/pages/en/blog/vulnerability/march-2025-ci-incident.md
Outdated
Show resolved
Hide resolved
|
FYI for Node.js Website Team (Unrelated to contents of this PR): |
|
Lighthouse Results
|
apps/site/pages/en/blog/vulnerability/march-2025-ci-incident.md
Outdated
Show resolved
Hide resolved
apps/site/pages/en/blog/vulnerability/march-2025-ci-incident.md
Outdated
Show resolved
Hide resolved
Codecov ReportAll modified and coverable lines are covered by tests ✅
✅ All tests successful. No failed tests found. Additional details and impacted files@@ Coverage Diff @@
## main #7658 +/- ##
=======================================
Coverage ? 74.61%
=======================================
Files ? 96
Lines ? 7689
Branches ? 192
=======================================
Hits ? 5737
Misses ? 1950
Partials ? 2 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
FYI, I might not be available next week. Feel free to merge this PR on Monday. |
mcollina
left a comment
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.
I need to send some edit, will do asap.
apps/site/pages/en/blog/vulnerability/march-2025-ci-incident.md
Outdated
Show resolved
Hide resolved
Co-authored-by: Antoine du Hamel <[email protected]> Signed-off-by: Matteo Collina <[email protected]>
apps/site/pages/en/blog/vulnerability/march-2025-ci-incident.md
Outdated
Show resolved
Hide resolved
Co-authored-by: Antoine du Hamel <[email protected]> Signed-off-by: Matteo Collina <[email protected]>
Signed-off-by: Matteo Collina <[email protected]>
mhdawson
left a comment
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.
LGTM
|
@nodejs/nodejs-website can someone help me sorting out the linting here? This is exactly like we want it. |
mcollina
left a comment
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.
lgtm
apps/site/pages/en/blog/vulnerability/march-2025-ci-incident.md
Outdated
Show resolved
Hide resolved
Signed-off-by: Matteo Collina <[email protected]>
apps/site/pages/en/blog/vulnerability/march-2025-ci-incident.md
Outdated
Show resolved
Hide resolved
Signed-off-by: Matteo Collina <[email protected]>
You need to run
What does |
Note that the content is correct. |
|
Can you run prettier on the content? The ESLint issue is just a warning, and wouldn't fail the quality checks |
Signed-off-by: Matteo Collina <[email protected]>
mcollina
left a comment
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.
lgtm
cc: @nodejs/tsc