-
Notifications
You must be signed in to change notification settings - Fork 435
feat: add a --prodIfUnlocked flag #1988
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
Conversation
This lets you publish to production if it's unlocked in the Netlfify dashboard and create drafts otherwise. closes #1392 The oclif/command lib doesn't let us create a `--prod=auto` flag that's also a boolean flag so it's separate. I think it's clearer anyway.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @kasbah for closing your own issue :)
For some reason I can't push changes to the PR, can you make sure to allow edits from maintainers?
To get the tests to pass you need to sync the docs as described here https://github.com/netlify/cli/blob/master/CONTRIBUTING.md#pull-requests.
Also, can you please use camelCase for the flag to be consistent with other flags?
|
Hey, thanks for reviewing. I've made the suggested changes. Not sure why you can't push to my branch or how best to enable it. Let me know if you still need access. |
This lets you publish to production if it's unlocked in the Netlfify dashboard and create drafts otherwise.
closes #1392
The oclif/command lib doesn't let us create a
--prod=autoflag that's also a boolean flag so it's separate. I think it's clearer anyway.