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

Add "--no-ignore-optional" for netlify #37

Merged
merged 2 commits into from
Feb 6, 2018

Conversation

futuregerald
Copy link
Contributor

This is currently necessary for Netlify since switching to yarn version 1.3.2 by default.

This is currently necessary for Netlify since switching to yarn version 1.3.2 by default.
@futuregerald
Copy link
Contributor Author

Netlify now uses Yarn version 1.3.2 by default for new sites using Yarn. This PR will add --no-ignore-optional flag which is necessary since otherwise some modules won't install and the build will fail.

@erquhart
Copy link
Contributor

erquhart commented Feb 6, 2018

@futuregerald what's really sad is I just spent like an hour figuring out what on earth was happening with this template and was just about to open this exact PR lol. Thanks for this!

@erquhart erquhart merged commit 3bd842c into decaporg:master Feb 6, 2018
csw1995 pushed a commit to csw1995/unused-blog that referenced this pull request Aug 14, 2019
Add "--no-ignore-optional" for netlify
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