Skip to content

Conversation

@berezovskyi
Copy link
Member

Updated concurrency group to include lyoRepo and lyoBranch inputs.

Updated concurrency group to include lyoRepo and lyoBranch inputs.
Copilot AI review requested due to automatic review settings December 22, 2025 11:17
Copy link

Copilot AI left a 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 updates the concurrency group configuration in the GitHub Actions workflow to include lyoRepo and lyoBranch inputs, allowing better isolation of concurrent workflow runs based on these parameters.

  • Modified concurrency group identifier to include workflow-specific inputs
  • Added references to lyoRepo and lyoBranch inputs in the concurrency group string

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.


concurrency:
group: ${{ github.workflow }}-${{ github.ref }}
group: ${{ github.workflow }}-${{ github.ref }}-${{ github.event.inputs.lyoRepo }}-${{ github. event.inputs.lyoBranch }}
Copy link

Copilot AI Dec 22, 2025

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 is an extra space between "github." and "event.inputs.lyoBranch" which will cause the workflow to fail. The expression should be "github.event.inputs.lyoBranch" without any space.

Suggested change
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 }}

Copilot uses AI. Check for mistakes.

concurrency:
group: ${{ github.workflow }}-${{ github.ref }}
group: ${{ github.workflow }}-${{ github.ref }}-${{ github.event.inputs.lyoRepo }}-${{ github. event.inputs.lyoBranch }}
Copy link

Copilot AI Dec 22, 2025

Choose a reason for hiding this comment

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

The concurrency group references "github.event.inputs.lyoRepo" but this input is not defined in the workflow_dispatch inputs section. Only "lyoBranch" and "link" inputs are defined. Either add the "lyoRepo" input definition or remove it from the concurrency group.

Suggested change
group: ${{ github.workflow }}-${{ github.ref }}-${{ github.event.inputs.lyoRepo }}-${{ github. event.inputs.lyoBranch }}
group: ${{ github.workflow }}-${{ github.ref }}-${{ github.event.inputs.lyoBranch }}

Copilot uses AI. Check for mistakes.
@berezovskyi berezovskyi merged commit 20c2366 into main Dec 22, 2025
4 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.

2 participants