Skip to content

Conversation

@neilcook
Copy link
Contributor

@neilcook neilcook commented Dec 17, 2025

I created a tag, but it didn't trigger a workflow action. Probably because the branch: main excluded tag pushes. That restriction isn't necessary anyway, because we check for a release tag in the workflow.

@neilcook neilcook requested a review from Luit December 17, 2025 16:09
Comment on lines 4 to 6
push:
branches: [ "main" ]
pull_request:
branches: [ "main" ]

Copy link

Choose a reason for hiding this comment

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

Examples I've seen do this with on: [push, pull_request] or something similar. Not sure if that's any different. From a pure YAML perspective, I prefer the {} over null (which is what this does).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yeah, that might indeed be a bit more readable

Copy link
Contributor Author

Choose a reason for hiding this comment

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

done

@neilcook neilcook merged commit 80b6d6b into main Dec 18, 2025
5 checks passed
@neilcook neilcook deleted the ci-tag branch December 18, 2025 12:23
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