Skip to content

Conversation

@kasbah
Copy link
Contributor

@kasbah kasbah commented Mar 12, 2021

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.

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.
@kasbah kasbah requested a review from a team as a code owner March 12, 2021 18:49
@erezrokah erezrokah added the type: feature code contributing to the implementation of a feature and/or user facing functionality label Mar 13, 2021
Copy link
Contributor

@erezrokah erezrokah left a 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?

@kasbah kasbah changed the title feat: add a --prod-if-unlocked flag feat: add a --prodIfUnlocked flag Mar 15, 2021
@kasbah
Copy link
Contributor Author

kasbah commented Mar 15, 2021

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.

@erezrokah erezrokah enabled auto-merge (squash) March 15, 2021 12:24
@erezrokah erezrokah disabled auto-merge March 15, 2021 12:24
@erezrokah erezrokah enabled auto-merge (squash) March 15, 2021 12:24
@erezrokah erezrokah merged commit 750d8cc into netlify:master Mar 15, 2021
This was referenced Mar 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: feature code contributing to the implementation of a feature and/or user facing functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

A flag to publish if auto publish is on, create drafts otherwise

2 participants