Skip to content

Conversation

@iiio2
Copy link
Contributor

@iiio2 iiio2 commented Oct 6, 2025

I think for both vite and nuxt node v18 is deprecated now.

@iiio2 iiio2 requested a review from danielroe as a code owner October 6, 2025 16:39
@bolt-new-by-stackblitz
Copy link

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

@coderabbitai
Copy link

coderabbitai bot commented Oct 6, 2025

Walkthrough

Updates documentation at docs/1.getting-started/02.installation.md by changing the Node.js prerequisite example versions from (18, 20, etc) to (20, 22, etc), and adding a trailing period. No other files or logic are modified.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Single-file, textual documentation edit with minimal, homogeneous changes (version numbers and punctuation).

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Title Check ✅ Passed The title accurately summarises the update to the documentation to remove the deprecated Node v18 version and clearly conveys the main change without extraneous detail.
Description Check ✅ Passed The description states the rationale for removing Node v18 and directly relates to the documentation update in the changeset.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between d32859d and 2422147.

📒 Files selected for processing (1)
  • docs/1.getting-started/02.installation.md (1 hunks)

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Member

@danielroe danielroe left a comment

Choose a reason for hiding this comment

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

thanks!

@danielroe danielroe merged commit 7f48766 into nuxt:main Oct 7, 2025
8 of 9 checks passed
@github-actions github-actions bot mentioned this pull request Oct 7, 2025
@iiio2 iiio2 deleted the docs/remove-depticated-node branch October 8, 2025 02:21
@github-actions github-actions bot mentioned this pull request Oct 23, 2025
@github-actions github-actions bot mentioned this pull request Oct 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants