-
-
Notifications
You must be signed in to change notification settings - Fork 6.3k
fix: add redirection from /dist to /dist/ #5623
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.
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
Conversation
Signed-off-by: Ulises Gascón <[email protected]>
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
📦 Next.js Bundle Analysis for nodejs.orgThis analysis was generated by the Next.js Bundle Analysis action. 🤖 This PR introduced no changes to the JavaScript bundle! 🙌 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I changed the CF rule's regex to remove the trailing slash:
@targos I assume Caching Rules also need to be updated then? https://dash.cloudflare.com/07be8d2fbc940503ca1be344714cb0d1/nodejs.org/caching/cache-rules |
@ovflowd Updated as well. |
aaaargh, done :D |
Description
This is a simple redirection from
/dist
to/dist/
.Validation
Related Issues
Full context in nodejs/docker-node#1945.
Check List
npx turbo lint
to ensure the code follows the style guide. And runnpx turbo lint:fix
to fix the style errors if necessary.npx turbo format
to ensure the code follows the style guide.npx turbo test
to check if all tests are passing, and/ornpx turbo test:snapshot
to update snapshots if I created and/or updated React Components.