Skip to content

docs: fix link to vitest globals config #33702

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.

Sign up for GitHub

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

Jump to bottom
Merged
danielroe merged 1 commit into from
Nov 12, 2025
Merged

docs: fix link to vitest globals config #33702

danielroe merged 1 commit into from
Nov 12, 2025

Conversation

@pierreoa
Copy link
Contributor

@pierreoa pierreoa commented Nov 12, 2025

🔗 Linked issue

📚 Description

Not sure what changed but linking to https://vitest.dev/config/#globals doesn't properly resolve, https://vitest.dev/config/globals does.

@pierreoa pierreoa requested a review from danielroe as a code owner November 12, 2025 19:35
@bolt-new-by-stackblitz
Copy link

bolt-new-by-stackblitz bot commented Nov 12, 2025

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

@coderabbitai
Copy link

coderabbitai bot commented Nov 12, 2025 •
edited
Loading

Walkthrough

A hyperlink in the testing documentation was updated to point to a more specific URL path. The Vitest globals link in docs/1.getting-started/17.testing.md was changed from /config/#globals to /config/globals. This is a documentation-only modification with no content or logic changes.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

  • Single file modified with a straightforward hyperlink update
  • No logic or functional changes involved
  • Purely cosmetic documentation adjustment

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description check ✅ Passed The description is relevant to the changeset, explaining why the URL correction was necessary and identifying the specific documentation file affected.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check ✅ Passed The title accurately summarises the main change: fixing a link to Vitest globals configuration in documentation.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

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.

@danielroe danielroe changed the title docs: typo vitest url globals config in 17.testing.md docs: fix vitest url globals config Nov 12, 2025
@danielroe danielroe changed the title docs: fix vitest url globals config docs: fix link to vitest globals config Nov 12, 2025
@danielroe danielroe merged commit 07eccf3 into nuxt:main Nov 12, 2025
10 of 11 checks passed
@github-actions github-actions bot mentioned this pull request Nov 12, 2025
Merged
danielroe pushed a commit that referenced this pull request Dec 9, 2025
@pierreoa @danielroe
docs: fix link to vitest globals config (#33702)
583fed9
@github-actions github-actions bot mentioned this pull request Dec 9, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@danielroe danielroe Awaiting requested review from danielroe danielroe is a code owner

Assignees

No one assigned

Labels

4.x documentation

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@pierreoa @danielroe