Skip to content

Commit

Permalink
release.yml: Check pre_release once for all of homebrew-core jo…
Browse files Browse the repository at this point in the history
…b instead of once per each step of that job.

Partial mas-cli#638

Signed-off-by: Ross Goldberg <484615+rgoldberg@users.noreply.github.com>
  • Loading branch information
rgoldberg committed Nov 16, 2024
1 parent 30962b3 commit c91de70
Showing 1 changed file with 1 addition and 6 deletions.
7 changes: 1 addition & 6 deletions .github/workflows/release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -226,23 +226,19 @@ jobs:
homebrew-core:
runs-on: macos-15
needs: [start, prepare-release, homebrew-tap]
env:
PRE_RELEASE: ${{ needs.start.outputs.pre_release }}
if: ${{ needs.start.outputs.pre_release == 'false' }}
steps:
- uses: actions/checkout@v4
if: env.PRE_RELEASE == 'false'
with:
# A fetch-depth of 0 includes all history and tags for script/version
fetch-depth: 0
ref: ${{ needs.start.outputs.release_branch }}

- name: 👢 Bootstrap
if: env.PRE_RELEASE == 'false'
run: |
script/bootstrap -f
- name: 🔧 Configure Git Author
if: env.PRE_RELEASE == 'false'
run: |
git config --global user.name 'masclibot'
git config --global user.email 'masclibot@users.noreply.github.com'
Expand All @@ -253,7 +249,6 @@ jobs:
HOMEBREW_GITHUB_API_TOKEN: ${{ secrets.HOMEBREW_GITHUB_API_TOKEN }}
DRY_RUN: ${{ needs.start.outputs.dry_run }}
MAS_VERSION: ${{ needs.start.outputs.mas_version }}
if: env.PRE_RELEASE == 'false'
run: |
DRY_RUN=${DRY_RUN} \
script/brew_core_update ${MAS_VERSION}

0 comments on commit c91de70

Please sign in to comment.