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

feat: concurrency at GitHub workflow level #905

Merged
merged 5 commits into from
Mar 22, 2024

Conversation

zezke
Copy link
Contributor

@zezke zezke commented Mar 5, 2024

This PR adds a concurrency setting on workflow level, preventing multiple workflows run in parallel which our pipeline does not like. It also addressed the comments by @kaizencc in the incorrect older PR.

Fixes #915

@zezke zezke force-pushed the feat/workflow-concurrency branch from a2d2468 to ca633f5 Compare March 5, 2024 08:51
@zezke zezke force-pushed the feat/workflow-concurrency branch from 47f2fb5 to 680e155 Compare March 5, 2024 08:53
Copy link
Contributor

@kaizencc kaizencc left a comment

Choose a reason for hiding this comment

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

Thanks @zezke. Following the conversation on #914 I want to accept this change.

README.md Outdated Show resolved Hide resolved
src/workflows-model.ts Show resolved Hide resolved
src/workflows-model.ts Outdated Show resolved Hide resolved
@kaizencc kaizencc changed the title feat: expose concurrency at GitHub workflow level feat: concurrency at GitHub workflow level Mar 22, 2024
@mergify mergify bot merged commit 9fee514 into cdklabs:main Mar 22, 2024
12 checks passed
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.

Can't set concurrency
2 participants