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

odeprecated: handle deprecated and disabled formulae. #9542

Merged

Conversation

MikeMcQuaid
Copy link
Member

It's not useful to spend time complaining about or fixing deprecations or disables in deprecated or disabled formulae given we already complain on install and don't run them through CI.

Needed for #9403

@BrewTestBot
Copy link
Member

Review period will end on 2020-12-15 at 15:50:22 UTC.

@BrewTestBot BrewTestBot added the waiting for feedback Merging is blocked until sufficient time has passed for review label Dec 14, 2020
@MikeMcQuaid
Copy link
Member Author

@Homebrew/core Interested in your opinions on this. Another option would be applying this only to disable!d formula.

@fxcoudert
Copy link
Member

Sounds logical to me.

@MikeMcQuaid MikeMcQuaid added the critical Critical change which should be shipped as soon as possible. label Dec 15, 2020
@BrewTestBot BrewTestBot removed the waiting for feedback Merging is blocked until sufficient time has passed for review label Dec 15, 2020
@BrewTestBot
Copy link
Member

BrewTestBot commented Dec 15, 2020

Review period ended.

It's not useful to spend time complaining about or fixing deprecations
or disables in deprecated or disabled formulae given we already complain
on install and don't run them through CI.
@MikeMcQuaid MikeMcQuaid merged commit 34b9fe1 into Homebrew:master Dec 15, 2020
@MikeMcQuaid MikeMcQuaid deleted the odeprecated-deprecate-disable branch December 15, 2020 13:06
@BrewTestBot BrewTestBot added the outdated PR was locked due to age label Jan 15, 2021
@Homebrew Homebrew locked as resolved and limited conversation to collaborators Jan 15, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
critical Critical change which should be shipped as soon as possible. outdated PR was locked due to age
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants