Skip to content

docs(es): fix duplicated /es prefixes in localized links - #38

Merged
kattsushi merged 1 commit into
masterfrom
fix/docs-es-links
Feb 9, 2026
Merged

kattsushi merged 1 commit into
masterfrom
fix/docs-es-links

Conversation

@kattsushi

@kattsushi kattsushi commented Feb 9, 2026 •

Copy link
Copy Markdown
Contributor

chore: lint-staged dprint fmt uses --allow-no-files to avoid false failures

Summary by CodeRabbit

  • Documentation

    • Corrected documentation navigation links to ensure proper access across React, SolidJS, Backend, and Universal guides.
  • Chores

    • Updated code formatting tool configuration to improve build process robustness.

chore: lint-staged dprint fmt uses --allow-no-files to avoid false failures
@coderabbitai

coderabbitai Bot commented Feb 9, 2026 •

Copy link
Copy Markdown
Contributor
📝 Walkthrough

Walkthrough

This PR updates documentation navigation links in the Spanish docs by removing locale prefixes from internal link targets, and modifies the lint-staged configuration to allow the dprint formatter to execute when no files match the pattern.

Changes

Cohort / File(s) Summary
Documentation Link Updates
apps/docs/src/content/docs/es/index.mdx
Removed es/ prefix from internal navigation links across React, SolidJS, Backend, and Universal sections, updating paths to their non-prefixed equivalents.
Build Configuration
package.json
Updated dprint formatter command in lint-staged to include --allow-no-files flag, enabling execution when no files match the pattern.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

Poem

🐰 Links are hopping lighter now,
No more locale paths to bow,
Spanish docs take a cleaner stride,
While formatters handle files with pride,
Navigation maps the proper way!

🚥 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—fixing duplicated /es prefixes in Spanish documentation links as evidenced by the raw summary showing link updates in the es/index.mdx file.
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
  • Commit unit tests in branch fix/docs-es-links

No actionable comments were generated in the recent review. 🎉


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

@kattsushi
kattsushi merged commit 4218bee into master Feb 9, 2026
6 checks passed
kattsushi added a commit that referenced this pull request Mar 15, 2026
chore: lint-staged dprint fmt uses --allow-no-files to avoid false failures
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.

1 participant