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

Use deprecate! instead of discontinued #162720

Merged
merged 236 commits into from
Dec 18, 2023
Merged

Use deprecate! instead of discontinued #162720

merged 236 commits into from
Dec 18, 2023

Conversation

Rylan12
Copy link
Member

@Rylan12 Rylan12 commented Dec 17, 2023

See Homebrew/brew#16292

Move all instances of discontinued in the caveats stanza to a deprecate! call.

For now, I've left the date for all of these as today1 (2023-12-17) because it doesn't seem to me like it will be a good use of my time to go through each and figure out an appropriate date. If that isn't the best solution, though, I'm happy to hear about alternatives.

This PR was made using brew style --fix with Homebrew/brew#16351 checked out, and then the non-autocorrectable (when the caveats stanza includes more than just discontinued) offenses were handled manually.

Note: this PR cannot be merged until a new Homebrew/brew tag (4.2.0) is made.

Footnotes

  1. Except for dozer which I set to 2023-11-26 because there was a comment

@Rylan12 Rylan12 added the ci-syntax-only Only run syntax checks on CI. Use only for bulk changes. label Dec 17, 2023
@SMillerDev
Copy link
Member

Great work @Rylan12, although it's a shame I get to tick something off my list that I didn't do.

@chenrui333
Copy link
Member

Awesome job! Merging!

@chenrui333 chenrui333 merged commit b8ef1bc into Homebrew:master Dec 18, 2023
6 checks passed
@Rylan12 Rylan12 deleted the replace-discontinued-with-deprecate branch December 19, 2023 02:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge-skip ci-syntax-only Only run syntax checks on CI. Use only for bulk changes.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants