Skip to content

ci: trigger for releases now allows changes in major version#756

Merged
psafont merged 1 commit intomasterfrom
dev/pau/2026
Feb 19, 2026
Merged

ci: trigger for releases now allows changes in major version#756
psafont merged 1 commit intomasterfrom
dev/pau/2026

Conversation

@psafont
Copy link
Member

@psafont psafont commented Feb 19, 2026

The current pattern now is better at defining the shape of a release, it allows for some leeway at the end. Useful to add some string at the end, which is used when a patched release is made for XCP-ng.

See
https://docs.github.com/en/actions/reference/workflows-and-actions/workflow-syntax#filter-pattern-cheat-sheet

tags:
- '6.*'
# matches 0.0.0 through 99.99.99, allowing for suffixes
- '[0-9]?[0-9]+.[0-9]+[0-9]?.[0-9]+[0-9]?*'
Copy link
Collaborator

Choose a reason for hiding this comment

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

According to Copilot this is a glob pattern and not a regexp - so it that does not support + according to that.

Copy link
Collaborator

@lindig lindig Feb 19, 2026

Choose a reason for hiding this comment

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

You linked to the reference, so Copilot is wrong here.

Copy link
Collaborator

Choose a reason for hiding this comment

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

[0-9]+ should be enough to capture digits

Copy link
Collaborator

Choose a reason for hiding this comment

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

[0-9]+.[0-9]+.[0-9]+* imho is sufficient because * is a glob

Copy link
Member Author

Choose a reason for hiding this comment

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

Ah, you're right, + shouldn't be used at all, I want only to allow 1 or 2 digits here. I've changed the match

The current pattern now is better at defining the shape of a release, it
allows for some leeway at the end. Useful to add some string at the end,
which is used when a patched release is made for XCP-ng.

See
https://docs.github.com/en/actions/reference/workflows-and-actions/workflow-syntax#filter-pattern-cheat-sheet

Signed-off-by: Pau Ruiz Safont <pau.safont@vates.tech>
@psafont psafont merged commit 1650d9f into master Feb 19, 2026
7 checks passed
@psafont psafont deleted the dev/pau/2026 branch February 19, 2026 11:54
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.

3 participants