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

workflows/docker: base homebrew/brew on 22.04 #13218

Merged
merged 1 commit into from Apr 29, 2022
Merged

Conversation

Bo98
Copy link
Member

@Bo98 Bo98 commented Apr 28, 2022

As per discussion on Slack, I think we can make the switch now rather than wait some arbitrary length of time.

I've also disabled fail-fast so that if a image upload fails in the future, we don't stop the other uploads halfway unnecessarily.

Also disable fail-fast so that if a image upload fails we don't stop the other uploads halfway.
@BrewTestBot
Copy link
Member

Review period will end on 2022-04-29 at 20:27:43 UTC.

@BrewTestBot BrewTestBot added the waiting for feedback Merging is blocked until sufficient time has passed for review label Apr 28, 2022
@sjackman sjackman self-requested a review April 29, 2022 19:43
Copy link
Member

@sjackman sjackman left a comment

Choose a reason for hiding this comment

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

Thank you, Bo!

@BrewTestBot
Copy link
Member

Review period ended.

@BrewTestBot BrewTestBot removed the waiting for feedback Merging is blocked until sufficient time has passed for review label Apr 29, 2022
@Bo98 Bo98 merged commit b4f7dbc into master Apr 29, 2022
@Bo98 Bo98 deleted the docker-brew-22.04 branch April 29, 2022 21:03
@github-actions github-actions bot added the outdated PR was locked due to age label May 30, 2022
@github-actions github-actions bot locked as resolved and limited conversation to collaborators May 30, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
outdated PR was locked due to age
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants