-
Notifications
You must be signed in to change notification settings - Fork 5
Update maven-acceptance-manual.yml #484
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR adds concurrency control to the Maven acceptance test workflow to prevent multiple instances of the same workflow from running simultaneously. The concurrency configuration groups workflows by workflow name, git ref, Lyo repository, and Lyo branch, with automatic cancellation of in-progress runs for non-main/master branches.
Key Changes
- Added concurrency control block with group key based on workflow, ref, and input parameters
- Configured cancel-in-progress behavior to protect main/master branches while allowing cancellation on feature branches
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| default: 'n/a' | ||
|
|
||
| concurrency: | ||
| group: ${{ github.workflow }}-${{ github.ref }}-${{ github.event.inputs.lyoRepo }}-${{ github. event.inputs.lyoBranch }} |
Copilot
AI
Dec 22, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There is a syntax error in the GitHub Actions expression. There's an extra space between 'github.' and 'event' that will cause the workflow to fail. The expression should be github.event.inputs.lyoBranch (without the space).
| group: ${{ github.workflow }}-${{ github.ref }}-${{ github.event.inputs.lyoRepo }}-${{ github. event.inputs.lyoBranch }} | |
| group: ${{ github.workflow }}-${{ github.ref }}-${{ github.event.inputs.lyoRepo }}-${{ github.event.inputs.lyoBranch }} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
No description provided.