Skip to content

nit: formatting of import statement in i18n guide#6611

Closed
Luzefiru wants to merge 2 commits intoTanStack:mainfrom
Luzefiru:patch-1
Closed

nit: formatting of import statement in i18n guide#6611
Luzefiru wants to merge 2 commits intoTanStack:mainfrom
Luzefiru:patch-1

Conversation

@Luzefiru
Copy link

@Luzefiru Luzefiru commented Feb 7, 2026

Just a small formatting/typo nitpick.

Summary by CodeRabbit

  • Documentation
    • Corrected a code sample in the internationalization guide to fix import syntax and JSX/TSX formatting for clearer, valid examples.

@github-actions github-actions bot added the documentation Everything documentation related label Feb 7, 2026
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 7, 2026

📝 Walkthrough

Walkthrough

A documentation fix that corrects a TSX code sample in the internationalization guide by adding a missing semicolon and reformatting the JSX/HTML tag placement for proper syntax.

Changes

Cohort / File(s) Summary
Documentation
docs/router/framework/react/guide/internationalization-i18n.md
Adjusted TSX code sample formatting: added a missing semicolon to an import and moved the HTML/JSX tag onto its own line to ensure valid TSX syntax.

Sequence Diagram(s)

(omitted — change is a documentation/formatting fix and does not introduce multi-component control flow)

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • docs: add i18n guide #6219: Similar fix touching the same internationalization documentation, aligning TSX example formatting and HTML lang attribute usage.

Poem

🐰 A semicolon hopped back to the line,
I nudged the tag to stand alone and fine,
Now JSX breathes without a frown,
The docs wear syntax like a crown,
Hooray — small fixes, big delight! 🎉

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately describes the main change: a formatting adjustment to an import statement in the internationalization guide documentation.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

No actionable comments were generated in the recent review. 🎉


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.

@nx-cloud
Copy link

nx-cloud bot commented Feb 8, 2026

View your CI Pipeline Execution ↗ for commit f095fd3

Command Status Duration Result
nx run-many --target=build --exclude=examples/*... ✅ Succeeded 4s View ↗
nx affected --targets=test:eslint,test:unit,tes... ✅ Succeeded <1s View ↗

☁️ Nx Cloud last updated this comment at 2026-02-08 00:34:52 UTC


```tsx
import { getLocale } from '../paraglide/runtime'

Copy link
Contributor

Choose a reason for hiding this comment

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

so you just wanted to add a blank line?

@Luzefiru Luzefiru closed this Feb 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Everything documentation related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants