Support auto-merge for Cask updates #93
Conversation
.github/workflows/casks.yml
Outdated
| # Status check that is required in branch protection rules. | ||
| cask-status: |
There was a problem hiding this comment.
After enabling this in the repo's branch rules, the auto-merge step could work well.
.github/workflows/casks.yml
Outdated
| changed-casks: | ||
| name: Get changed casks | ||
| runs-on: ubuntu-latest | ||
| outputs: | ||
| matrix: ${{ steps.changed-files.outputs.all_changed_files }} | ||
| steps: | ||
| - uses: actions/checkout@v5 | ||
| - name: Get changed files | ||
| id: changed-files | ||
| uses: tj-actions/changed-files@v47 |
There was a problem hiding this comment.
https://github.com/mdogan/homebrew-zulu/actions/runs/18127831550
The action tj-actions/changed-files@v47 is not allowed in mdogan/homebrew-zulu because all actions must be from a repository owned by mdogan, created by GitHub, verified in the GitHub Marketplace, or match one of the patterns: peter-evans/create-pull-request@, tibdex/github-app-token@.
This can't work for now due to GitHub's limitations.
There was a problem hiding this comment.
You can view the results in
There was a problem hiding this comment.
I read some problems about tj-actions/changed-files action, that's why I'm hesitating to allow it.
There was a problem hiding this comment.
It works well, from my observations. And I noticed there are other solutions like the Homebrew official used, but they are too complex!
There was a problem hiding this comment.
Tested the auto-merge in Goooler#93.
FYI, I added a related PR to my repo. Goooler/homebrew-repo#61
There was a problem hiding this comment.
Also, if we want to land something like #50, this is useful.
Goooler
changed the title
|
This should be useful for the PRs like #94. |
Goooler
force-pushed
the
master
branch
5 times, most recently
from
fbf82be to
4e37cab
Compare
Goooler
force-pushed
the
master
branch
2 times, most recently
from
8ba4394 to
4495e9c
Compare
2 participants
Refs
One more step on #81.