Skip to content

fix(blog): correct codemod references in v22-to-v24 migration guide - #9095

Open
blue2cat wants to merge 1 commit into
nodejs:mainfrom
blue2cat:node-22-24-guide-2
Open

fix(blog): correct codemod references in v22-to-v24 migration guide#9095
blue2cat wants to merge 1 commit into
nodejs:mainfrom
blue2cat:node-22-24-guide-2

Conversation

@blue2cat

Copy link
Copy Markdown
Contributor

Description

After #9082 was merged, I noted several other issues in this post.

  • Rename http2-priority-removal to http2-priority-signaling to match the published codemod name
  • Rename the fs-truncate-to-ftruncate heading to fs-truncate-fd-deprecation. This makes it consistent with the codemod name.
  • Fix typos

Validation

Related Issues

Check List

  • I have read the Contributing Guidelines and made commit messages that follow the guideline.
  • I have run pnpm format to ensure the code follows the style guide.
  • I have run pnpm test to check if all tests are passing.
  • I have run pnpm build to check if the website builds without errors.
  • I've covered new added functionality with unit tests if necessary.

Copilot AI lite review requested due to automatic review settings August 10, 2026 23:22
@vercel

vercel Bot commented Aug 10, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
nodejs-org Ready Ready Preview Aug 10, 2026 11:22pm

Request Review

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the Node.js v22→v24 migration guide to ensure codemod references align with the published recipe/registry names and improves copy clarity so users can successfully locate and run the intended codemods.

Changes:

  • Corrects codemod naming to match published recipes (e.g., http2-priority-signaling, fs-truncate-fd-deprecation).
  • Adds/aligns source code and Codemod Registry references for consistency.
  • Fixes minor wording/typos that could confuse readers when following the guide.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@AugustinMauroy

Copy link
Copy Markdown
Member

Thanks for catching that seem copy pasta or idk form my part

@codecov

codecov Bot commented Aug 11, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 75.14%. Comparing base (79dafaa) to head (48ec82e).
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #9095      +/-   ##
==========================================
+ Coverage   75.13%   75.14%   +0.01%     
==========================================
  Files         102      102              
  Lines        9114     9114              
  Branches      376      376              
==========================================
+ Hits         6848     6849       +1     
+ Misses       2262     2261       -1     
  Partials        4        4              

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

@github-actions

Copy link
Copy Markdown
Contributor

📦 Build Size Comparison

Summary

Metric Value
Old Total First Load JS 7.16 MB
New Total First Load JS 7.16 MB
Delta 0 B (0.00%)

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.

4 participants