Skip to content

do not throw error when re-deploying same commit #14

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

Closed
wants to merge 1 commit into from

Conversation

urbany
Copy link

@urbany urbany commented Jul 21, 2021

Hi, some times on my CI/CD flow I deploy the same version (same commit) multiple times and sentry-cli seems to throw an error when we try to re-create a version with no commit changes. This flag ignores that and keeps sentry-cli from erroring allowing the deploy to continue.

Hi, some times on my CI/CD flow I deploy the same version (same commit) multiple times and sentry-cli seems to throw an error when we try to re-create a version with no commit changes. This flag ignores that and keeps sentry-cli from erroring allowing the deploy to continue.
@Exelord
Copy link
Member

Exelord commented Jul 22, 2021

Thanks! We would need to adjust the tests :)

@kevinkucharczyk
Copy link
Contributor

@urbany @Exelord was there any movement on this? We, too, would like to allow redeploying the same commit multiple times.

@urbany
Copy link
Author

urbany commented Oct 26, 2021

hi @kevinkucharczyk I didn't have time to look at the tests, I have been using my fork in production, but I do not recomend you do that. Instead, if you have time, check with @Exelord what needs to be done wit the tests and make a new PR.

@xomaczar
Copy link
Contributor

xomaczar commented Oct 26, 2021

Hey I just submitted a PR with the same functionality and passing tests (didn't see this PR). I don't care which PR gets merged, I would just like to have this functionality merged upstream and released.

@urbany
Copy link
Author

urbany commented Oct 27, 2021

@xomaczar I don't care which one gets merged either :) if @Exelord is ok with merging yours I'll close this

@Exelord
Copy link
Member

Exelord commented Oct 28, 2021

Done :) #18 merged!

@Exelord Exelord closed this Oct 28, 2021
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.

4 participants