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

deployment-pr.yml: do not deploy the same PR at the same time #4069

Merged
merged 1 commit into from Nov 12, 2023

Conversation

BacLuc
Copy link
Contributor

@BacLuc BacLuc commented Nov 10, 2023

Else the second deployment fails if you push to fast to the same PR.

See #4068 to see the effect

grafik
https://github.com/ecamp/ecamp3/actions/runs/6834522334
grafik

@carlobeltrame carlobeltrame temporarily deployed to pr4068 November 10, 2023 21:44 Destroyed
@carlobeltrame carlobeltrame temporarily deployed to pr4068 November 11, 2023 12:13 Destroyed
Else the second deployment fails if you push to fast
to the same PR.
The default behaviour is that the previous deployment is canceled.
@BacLuc BacLuc mentioned this pull request Nov 11, 2023
@carlobeltrame carlobeltrame temporarily deployed to pr4068 November 11, 2023 13:30 Destroyed
@BacLuc BacLuc added this pull request to the merge queue Nov 12, 2023
Merged via the queue into devel with commit 4d4114f Nov 12, 2023
49 checks passed
@BacLuc BacLuc deleted the cd-serialize-deployment branch November 14, 2023 08:27
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.

None yet

3 participants