Skip to content

Commit

Permalink
add required approval and merge instead of mark as auto merge
Browse files Browse the repository at this point in the history
  • Loading branch information
burnettk committed Feb 5, 2024
1 parent e037365 commit 347ef3e
Showing 1 changed file with 9 additions and 2 deletions.
11 changes: 9 additions & 2 deletions .github/workflows/auto-merge-dependabot-prs.yml
Original file line number Diff line number Diff line change
Expand Up @@ -73,7 +73,14 @@ jobs:

- name: print pr number
run: cat pr_number
- name: actually merge it
run: gh pr merge --auto --merge "$(cat pr_number)"
# the repo requires one approval. if a dependabot change passes tests, that is good enough.
- name: approve pr
run: gh pr review --approve "$(cat pr_number)"
env:
GITHUB_TOKEN: ${{secrets.GITHUB_TOKEN}}
# if the merge --auto flag were added, and if the repo allowed it at https://github.com/sartography/spiff-arena/settings,
# it would set up the pr to auto merge when all requirements were met. but we just want to merge now.
- name: set up pr to auto merge when all requirements are met
run: gh pr merge --squash "$(cat pr_number)"
env:
GITHUB_TOKEN: ${{secrets.GITHUB_TOKEN}}

0 comments on commit 347ef3e

Please sign in to comment.