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.

Sign up for GitHub

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

Jump to bottom

formulary: use default bottle root_url. #14719

Merged
merged 1 commit into from Feb 20, 2023
Merged

formulary: use default bottle root_url. #14719

merged 1 commit into from Feb 20, 2023

Conversation

MikeMcQuaid
Copy link
Member

We never change this value in formula.json so ignore it for security.

@MikeMcQuaid
formulary: use default bottle root_url.
14a11f5 
We never change this value in `formula.json` so ignore it for security.
@MikeMcQuaid MikeMcQuaid added the critical Critical change which should be shipped as soon as possible. label Feb 20, 2023
@BrewTestBot
Copy link
Member

Review period skipped due to critical label.

@MikeMcQuaid MikeMcQuaid merged commit 6b7ecd1 into Homebrew:master Feb 20, 2023
@MikeMcQuaid MikeMcQuaid deleted the no_api_root_url branch February 20, 2023 10:21
Rylan12
Rylan12 reviewed Feb 20, 2023
View reviewed changes
Copy link
Member

@Rylan12 Rylan12 left a comment

Choose a reason for hiding this comment

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

Thanks, @MikeMcQuaid!

@github-actions github-actions bot added the outdated PR was locked due to age label Mar 23, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 23, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@Rylan12 Rylan12 Rylan12 left review comments

@BrewTestBot BrewTestBot BrewTestBot approved these changes

Assignees
No one assigned
Labels
critical Critical change which should be shipped as soon as possible. outdated PR was locked due to age
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@MikeMcQuaid @BrewTestBot @Rylan12