-
Notifications
You must be signed in to change notification settings - Fork 925
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
Make CI run all branches on push #5836
Conversation
We create branches for patch versions that need to be verified by CI.
I verified that CI ran on non-main branches. https://github.com/ikhoon/armeria/actions/runs/10107663381 |
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.
👍 👍 👍
branches: | ||
- main | ||
- "**" |
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.
Question) Is this equivalent to if we just remove these two lines altogether?
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.
I tried this setting on my forked repo first but it didn't seem to work.
on:
push:
tags-ignore:
# The release versions will be verified by 'publish-release.yml'
- armeria-*
pull_request:
merge_group:
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.
👍 👍 👍
Thanks for the quick review. |
Added the same change to Central dogma: |
We create branches for patch versions that need to be verified by CI. Reference: https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#filter-pattern-cheat-sheet > '**' : Matches all branch and tag names. This is the default behavior when you don't use a branches or tags filter.
CI for 1.29.3 is now running. https://github.com/line/armeria/actions/runs/10107942116 |
We create branches for patch versions that need to be verified by CI.
Reference: https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#filter-pattern-cheat-sheet