Skip to content

fix: concurrency fixes for release-drafter and deploy-site workflows #629

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

Merged
merged 4 commits into from
Jul 4, 2025

Conversation

vishalvivekm
Copy link

Notes for Reviewers

This PR assignes different concurrency groups to release-drafter and deploy workflows to ensure they can run independently, at the same time.

Release Drafter:

  • ignore changes made to .github or archive/ (cloud releases) folders.
  • only one workflow runs at a time, latest one cancels previous runs

Pages deployment:

  • only one workflow runs at a time; latest incoming run cancels any running workflow. Ensures we always deploy the latest changes rather than outdated code.

Behaviour:

Workflow A arrives → finds nothing in its concurrency group → starts running
Workflow B arrives → finds job A running in its concurrency group → kills A → starts running

Signed commits

  • Yes, I signed my commits.

Copy link

netlify bot commented Jul 4, 2025

Deploy Preview for bejewelled-pegasus-b0ce81 ready!

Name Link
🔨 Latest commit 3f05a48
🔍 Latest deploy log https://app.netlify.com/projects/bejewelled-pegasus-b0ce81/deploys/6867e6b51d8f880008408a81
😎 Deploy Preview https://deploy-preview-629--bejewelled-pegasus-b0ce81.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@vishalvivekm vishalvivekm requested a review from zihanKuang July 4, 2025 13:48
@vishalvivekm vishalvivekm changed the title concurrency fixes for release-drafter and deploy-site workflow workflow fix: concurrency fixes for release-drafter and deploy-site workflows Jul 4, 2025
@vishalvivekm vishalvivekm merged commit 5a928df into master Jul 4, 2025
5 of 6 checks passed
@vishalvivekm vishalvivekm deleted the vishalvivekm-patch-1 branch July 4, 2025 14:35
@@ -5,6 +5,12 @@ on:
# our release branch
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we need the release-drafter workflow? If "no", please proof the the release notes included in the Cloud releases and the Kanvas releases are sourced elsewhere,

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants