-
-
Notifications
You must be signed in to change notification settings - Fork 8.7k
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
fix(v2): remove webpackConfig.resolve.symlinks: true #5126
Conversation
✔️ [V2] 🔨 Explore the source changes: 05dcd1f 🔍 Inspect the deploy log: https://app.netlify.com/sites/docusaurus-2/deploys/60e723004c10270009886b85 😎 Browse the preview: https://deploy-preview-5126--docusaurus-2.netlify.app |
⚡️ Lighthouse report for the changes in this PR:
Lighthouse ran on https://deploy-preview-5126--docusaurus-2.netlify.app/ |
Size Change: +1.24 kB (0%) Total Size: 696 kB
ℹ️ View Unchanged
|
… as docs source is possible
Any chance this can be reapplied? |
Do you mean setting true or false? Because we reverted this change in #5164, but now we could try again to reapply this PR's changes But why do you care, considering you can easily change this setting yourself by using a plugin + |
I mean to have |
Motivation
remove
webpackConfig.resolve.symlinks: true
This might not be intuitive, but to make Docusaurus work with symlinked folders, we have to use default/false instead.
Should fix #3272
Have you read the Contributing Guidelines on pull requests?
yes
Test Plan
tests + preview