Skip to content
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 docs:build #720

Merged
merged 3 commits into from
Feb 2, 2023
Merged

Fix docs:build #720

merged 3 commits into from
Feb 2, 2023

Conversation

aristidesstaffieri
Copy link
Contributor

It looks like our docs build was failing because of a bug that's been in terser-webpack-plugin for a while, which is consumed by docusaurus in newer versions.

Details -
facebook/docusaurus#3657
facebook/docusaurus#3515

Solution -
Pin terser-webpack-plugin to the version before this was introduced until it is resolved in future versions by setting a specific version in the top level yarn resolution.

@aristidesstaffieri
Copy link
Contributor Author

Hmm, still lands in a failing state but no obvious error in the logs anymore. Still looking into this on the Netlify side, seem ok locally.

@piyalbasu
Copy link
Contributor

@aristidesstaffieri Ah, I see the issue - it's using an old version of Node (12) rather than the one it expects. I'll show you how to upgrade

Copy link
Contributor

@piyalbasu piyalbasu left a comment

Choose a reason for hiding this comment

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

🎉 🎉 🎉

@aristidesstaffieri aristidesstaffieri merged commit 775855f into master Feb 2, 2023
@aristidesstaffieri aristidesstaffieri deleted the fix-docasaurus-build branch February 2, 2023 22:10
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.

2 participants