Skip to content

Configure cluster management concurrency in a consistent place#163

Merged
danielfrankcom merged 1 commit intomainfrom
fix/cm-concurrency-consistency
May 29, 2025
Merged

Configure cluster management concurrency in a consistent place#163
danielfrankcom merged 1 commit intomainfrom
fix/cm-concurrency-consistency

Conversation

@danielfrankcom
Copy link
Copy Markdown
Contributor

This PR is a cosmetic change to the structure of the cluster management .yml files.

The concurrency blocks were added in #149, but while working on #162 I noticed they should really be grouped with the job configuration at the top. This was done inconsistently in #149 for some reason, with the .NET sample already grouping it at the top of the job configuration.

Having the concurrency block at the top makes it a bit easier to spot given the steps for the job are variable length and sometimes quite complex.

By submitting this pull request, I confirm that my contribution is made under the terms of the MIT-0 license.

@danielfrankcom danielfrankcom requested a review from vic-tsang May 29, 2025 16:08
@danielfrankcom danielfrankcom merged commit 08eed41 into main May 29, 2025
17 checks passed
@danielfrankcom danielfrankcom deleted the fix/cm-concurrency-consistency branch May 29, 2025 16:28
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