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(create-gatsby): Use legacy peer deps #29856

Merged
merged 2 commits into from
Mar 1, 2021
Merged

Conversation

ascorbic
Copy link
Contributor

@ascorbic ascorbic commented Mar 1, 2021

Two npm 7 fixes. Passes --legacy-peer-deps to npm when setting-up, to align behaviour across versions. Also formats package.json, as installing no longer does

@gatsbot gatsbot bot added the status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer label Mar 1, 2021
@ascorbic ascorbic added topic: cli Related to the Gatsby CLI and removed status: triage needed Issue or pull request that need to be triaged and assigned to a reviewer labels Mar 1, 2021
LekoArts
LekoArts previously approved these changes Mar 1, 2021
@ascorbic ascorbic added the bot: merge on green Gatsbot will merge these PRs automatically when all tests passes label Mar 1, 2021
@ascorbic ascorbic merged commit a92b276 into master Mar 1, 2021
@ascorbic ascorbic deleted the fix/init-legacy-peer-deps branch March 1, 2021 11:56
vladar pushed a commit that referenced this pull request Mar 1, 2021
* fix(create-gatsby): Use legacy peer deps

* Update tests

(cherry picked from commit a92b276)
vladar pushed a commit that referenced this pull request Mar 1, 2021
* fix(create-gatsby): Use legacy peer deps

* Update tests

(cherry picked from commit a92b276)

Co-authored-by: Matt Kane <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bot: merge on green Gatsbot will merge these PRs automatically when all tests passes topic: cli Related to the Gatsby CLI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants