Skip to content

Commit

Permalink
workflows/autopublish: remove
Browse files Browse the repository at this point in the history
This has been disabled for a month now, and it doesn't seem like we
missed it too much. I think it's safe to remove this now.

Also, now that we no longer attempt to merge approved pull requests at
regular intervals, there is no need to dismiss approvals from PRs when
this workflow fails. Presumably this was needed to avoid the workflow
from repeatedly attempting to publish a PR that cannot be published, but
this no longer a concern.
  • Loading branch information
carlocab committed May 31, 2023
1 parent 159de1a commit cd2ab8a
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 44 deletions.
35 changes: 0 additions & 35 deletions .github/workflows/autopublish.yml

This file was deleted.

10 changes: 1 addition & 9 deletions .github/workflows/publish-commit-bottles.yml
Original file line number Diff line number Diff line change
Expand Up @@ -331,7 +331,7 @@ jobs:
working-directory: ${{steps.set-up-homebrew.outputs.repository-path}}

- name: Post comment on failure
if: ${{!success()}}
if: failure()
uses: Homebrew/actions/post-comment@master
with:
token: ${{secrets.GITHUB_TOKEN}}
Expand All @@ -340,14 +340,6 @@ jobs:
bot_body: ":warning: Bottle publish [failed](${{env.RUN_URL}}). CC @carlocab"
bot: github-actions[bot]

- name: Dismiss approvals on failure
if: failure()
uses: Homebrew/actions/dismiss-approvals@master
with:
token: ${{secrets.GITHUB_TOKEN}}
pr: ${{inputs.pull_request}}
message: "bottle publish failed"

- name: Wait until pull request branch is in sync with local repository
id: wait-until-in-sync
working-directory: ${{steps.set-up-homebrew.outputs.repository-path}}
Expand Down

0 comments on commit cd2ab8a

Please sign in to comment.