Skip to content
This repository has been archived by the owner on May 2, 2024. It is now read-only.

Use deprecate! instead of discontinued #18724

Merged
merged 60 commits into from
Dec 18, 2023
Merged

Use deprecate! instead of discontinued #18724

merged 60 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 today (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.

@Rylan12 Rylan12 added the ci-syntax-only Only run syntax checks on CI. Use only for bulk changes. label Dec 17, 2023
@p-linnane p-linnane merged commit a7508b9 into Homebrew:master Dec 18, 2023
9 checks passed
@Rylan12 Rylan12 deleted the replace-discontinued-with-deprecate branch December 19, 2023 02:32
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 19, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
automerge-skip ci-syntax-only Only run syntax checks on CI. Use only for bulk changes. outdated
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants