search

keystonejs / create-keystone-app

CLI app that makes it easy to get started with Keystone
https://keystonejs.com/tutorials/getting-started-with-create-keystone-next-app
MIT License
40 stars 9 forks source link

added --use-npm flag #290

Closed schoening closed 2 years ago

schoening commented 2 years ago

I have added a --use-npm flag that skips the yarn install attempt when running npx create-keystone-app

I tried to get rid of duplication and refactored it into their own functions. Please feel free to give me feedback if this structure is not to your liking!

changeset-bot[bot] commented 2 years ago

⚠️ No Changeset found

Latest commit: 96e6af122f3e54ffbed164620349a8b64b3a7911

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

dcousens commented 2 years ago

Thanks @schoening , but I think will reject this in favour of what is being discussed in https://github.com/keystonejs/create-keystone-app/issues/214 (removing the package manager specific parts)