Skip to content
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

Merged
merged 1 commit into from
Jul 26, 2024
Merged

Conversation

ikhoon
Copy link
Contributor

@ikhoon ikhoon commented Jul 26, 2024

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.

We create branches for patch versions that need to be verified by CI.
@ikhoon ikhoon added this to the 1.30.0 milestone Jul 26, 2024
@ikhoon
Copy link
Contributor Author

ikhoon commented Jul 26, 2024

I verified that CI ran on non-main branches. https://github.com/ikhoon/armeria/actions/runs/10107663381

Copy link
Contributor

@jrhee17 jrhee17 left a comment

Choose a reason for hiding this comment

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

👍 👍 👍

Comment on lines 4 to +5
branches:
- main
- "**"
Copy link
Contributor

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?

Copy link
Contributor Author

@ikhoon ikhoon Jul 26, 2024

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:

Copy link
Contributor

@minwoox minwoox left a comment

Choose a reason for hiding this comment

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

👍 👍 👍

minwoox added a commit to minwoox/centraldogma that referenced this pull request Jul 26, 2024
@ikhoon
Copy link
Contributor Author

ikhoon commented Jul 26, 2024

Thanks for the quick review.

@ikhoon ikhoon merged commit 5393507 into line:main Jul 26, 2024
15 checks passed
@ikhoon ikhoon deleted the ci-run-all-branches branch July 26, 2024 08:00
@minwoox
Copy link
Contributor

minwoox commented Jul 26, 2024

Added the same change to Central dogma:
https://github.com/line/centraldogma/pull/996/files#diff-aeca80f8efee750eeb1db39a3fbb3a4260d269a55a4f77da15126330fe26d4beR5

ikhoon added a commit that referenced this pull request Jul 26, 2024
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.
@ikhoon
Copy link
Contributor Author

ikhoon commented Jul 26, 2024

CI for 1.29.3 is now running. https://github.com/line/armeria/actions/runs/10107942116

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants