-
-
Notifications
You must be signed in to change notification settings - Fork 6.5k
fix: images for node 18 eol blog post #7856
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
Signed-off-by: Trivikram Kamat <16024985+trivikr@users.noreply.github.com>
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
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 corrects the order of the images in the Node.js 18 EOL blog post so that the release schedule appears before the EOL graph.
- Swapped the image reference definitions to show the release schedule first, followed by the EOL graph.
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 #7856 +/- ##
==========================================
+ Coverage 75.44% 75.48% +0.03%
==========================================
Files 101 101
Lines 8305 8305
Branches 218 218
==========================================
+ Hits 6266 6269 +3
+ Misses 2037 2034 -3
Partials 2 2 ☔ View full report in Codecov by Sentry. |
|
Fast track requested |
|
Lighthouse Results
|
Description
The images are not accurate on Node.js 18 EOL blog post.
The release schedule image should appear first, followed by EOL Node.js graph.
Validation
Check preview at https://nodejs-org-git-trivikr-patch-1-openjs.vercel.app/en/blog/announcements/node-18-eol-support
Related Issues
N/A
Check List
pnpm formatto ensure the code follows the style guide.pnpm testto check if all tests are passing.pnpm buildto check if the website builds without errors.