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

Disable --create on push by default #1

Closed
wants to merge 1 commit into from

Conversation

emcfarlane
Copy link
Collaborator

Push by default sets the flag --create to automatically create the repository if it doesn't exist. This behaviour, however, requires elevated permissions not usually present in CI. In most cases this won't be needed as the modules will already be created. Therefore to cover the common case we now change the default input for push_create to false. To maintain the current behaviour explicitly set push_create: true.

Push by default sets `--create` flag to automatically create the
repostory if it doesn't exist. This behaviour, however, requires
elevated permissions not usually present in CI. In most cases this won't
be needed as the modules will already be created. Therefore to cover the
common case we now change the default input for `push_create` to false.
To maintain the current behaviour explicitly set `push_create: true`.
Copy link

The latest Buf updates on your PR.

NameStatus
build✅ passed
lint⏩ skipped
format❌ failed
breaking✅ passed

@nicksnyder
Copy link
Member

Not going to merge this. We should allow writers permission to create repos.

@emcfarlane
Copy link
Collaborator Author

Closing, changing scope of writers to allow creating repos.

@emcfarlane emcfarlane closed this May 28, 2024
@emcfarlane emcfarlane deleted the ed/defaultPushCreate branch May 28, 2024 17:43
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