docs: vercel-redirect-checker #2967
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
… validation failures
Modified the pre-commit hook to pass only staged markdown files to markdownlint instead of running it on all files in the project. Changes: - Extract staged markdown files excluding docs/sdk/ - Pass filtered list directly to markdownlint via xargs - Consistent with Prettier formatting approach (staged files only) This ensures developers aren't blocked by markdown issues in files they're not modifying, while still validating all committed changes.
…warning TypeScript's tsc command requires full project context for accurate type checking and cannot be limited to only staged files. This change: - Adds SKIP_TS_CHECK environment variable to bypass the check - Warns users that TypeScript checks the entire project - Provides clear instructions when check fails on unrelated errors - Improves error messages to explain the project-wide scope Usage: - One-time skip: SKIP_TS_CHECK=1 git commit - Permanent skip: export SKIP_TS_CHECK=1 (add to shell profile) This prevents blocking commits due to unrelated TypeScript errors while still allowing teams to enforce TypeScript checking when desired.
…e scope warning" This reverts commit 9ca0617.
Vercel uses path-to-regexp v6.1.0 which requires specific syntax: - The ? modifier only applies to parameters (like :param?), not literals - Using /? at the end of paths is invalid syntax - Vercel handles trailing slashes automatically with strict: false Changes: - Removed /? suffix from all 423 redirect source patterns - Pattern /anytrust/? → /anytrust (now matches both /anytrust and /anytrust/) - Follows path-to-regexp v6.1.0 syntax requirements This fixes the error: "Redirect at index 1 has invalid source pattern /anytrust/?" Refs: https://vercel.com/docs/errors/error-list#invalid-route-source-pattern
Fixed 4 additional patterns that had ? suffix without /: - /sdk-docs/dataEntities/networks? → /sdk-docs/dataEntities/networks - /sdk/introduction? → /sdk/introduction - /sdk/utils_multicall? → /sdk/utils_multicall - /stylus/tools/stylus-cli? → /stylus/tools/stylus-cli These patterns were also invalid in path-to-regexp v6.1.0 syntax. Vercel dev now starts without any pattern validation errors.
Co-authored-by: Copilot <[email protected]>
Added "trailingSlash": false to vercel.json to ensure all URLs with trailing slashes are automatically redirected to their non-trailing-slash equivalents. This provides consistent behavior across all 2,000+ redirects without needing individual /? suffixes on each redirect pattern.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
…s/arbitrum-docs into disable-vercel-redirect-test
Redirect and format check fix
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
check-markdown.tsthat checks for staged deletions of markdown (.md or .mdx) files