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

Deprecations, disables, deletions for 3.4.0 #12913

Merged
merged 1 commit into from
Feb 24, 2022
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
4 changes: 2 additions & 2 deletions Library/Homebrew/cask/dsl/version.rb
Original file line number Diff line number Diff line change
Expand Up @@ -153,14 +153,14 @@ def after_comma
# @api public
sig { returns(T.self_type) }
def before_colon
# odeprecated "Cask::DSL::Version#before_colon", "Cask::DSL::Version#csv"
odeprecated "Cask::DSL::Version#before_colon", "Cask::DSL::Version#csv"
version { split(":", 2).first }
end

# @api public
sig { returns(T.self_type) }
def after_colon
# odeprecated "Cask::DSL::Version#after_colon", "Cask::DSL::Version#csv"
odeprecated "Cask::DSL::Version#after_colon", "Cask::DSL::Version#csv"
version { split(":", 2).second }
end

Expand Down
3 changes: 0 additions & 3 deletions Library/Homebrew/cli/named_args.rb
Original file line number Diff line number Diff line change
Expand Up @@ -109,9 +109,6 @@ def load_formula_or_cask(name, only: nil, method: nil, prefer_loading_from_api:
resolve_latest_keg(name)
when :default_kegs
resolve_default_keg(name)
when :keg
odisabled "`load_formula_or_cask` with `method: :keg`",
"`load_formula_or_cask` with `method: :default_kegs`"
when :kegs
_, kegs = resolve_kegs(name)
kegs
Expand Down
7 changes: 0 additions & 7 deletions Library/Homebrew/cmd/list.rb
Original file line number Diff line number Diff line change
Expand Up @@ -24,9 +24,6 @@ def list_args
description: "List only formulae, or treat all named arguments as formulae."
switch "--cask", "--casks",
description: "List only casks, or treat all named arguments as casks."
switch "--unbrewed",
description: "List files in Homebrew's prefix not installed by Homebrew.",
replacement: "`brew --prefix --unbrewed`"
switch "--full-name",
description: "Print formulae with fully-qualified names. Unless `--full-name`, `--versions` "\
"or `--pinned` are passed, other options (i.e. `-1`, `-l`, `-r` and `-t`) are "\
Expand Down Expand Up @@ -59,11 +56,7 @@ def list_args
conflicts "--pinned", "--multiple"
conflicts "--pinned", "--cask"
conflicts "--cask", "--multiple"
["--formula", "--cask", "--full-name", "--versions", "--pinned"].each do |flag|
conflicts "--unbrewed", flag
end
["-1", "-l", "-r", "-t"].each do |flag|
conflicts "--unbrewed", flag
conflicts "--versions", flag
conflicts "--pinned", flag
end
Expand Down
2 changes: 1 addition & 1 deletion Library/Homebrew/dev-cmd/bump-cask-pr.rb
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@ def bump_cask_pr_args
def bump_cask_pr
args = bump_cask_pr_args.parse

odeprecated "`brew bump-cask-pr --write`", "`brew bump-cask-pr --write-only`" if args.write?
odisabled "`brew bump-cask-pr --write`", "`brew bump-cask-pr --write-only`" if args.write?

# This will be run by `brew style` later so run it first to not start
# spamming during normal output.
Expand Down
4 changes: 0 additions & 4 deletions Library/Homebrew/dev-cmd/pr-automerge.rb
Original file line number Diff line number Diff line change
Expand Up @@ -26,10 +26,6 @@ def pr_automerge_args
description: "Pull requests do not require approval to be merged."
switch "--publish",
description: "Run `brew pr-publish` on matching pull requests."
switch "--autosquash",
description: "Instruct `brew pr-publish` to automatically reformat and reword commits "\
"in the pull request to our preferred format.",
replacement: "`--no-autosquash` to opt out"
switch "--no-autosquash",
description: "Instruct `brew pr-publish` to skip automatically reformatting and rewording commits "\
"in the pull request to the preferred format."
Expand Down
4 changes: 0 additions & 4 deletions Library/Homebrew/dev-cmd/pr-publish.rb
Original file line number Diff line number Diff line change
Expand Up @@ -16,10 +16,6 @@ def pr_publish_args
Publish bottles for a pull request with GitHub Actions.
Requires write access to the repository.
EOS
switch "--autosquash",
description: "If supported on the target tap, automatically reformat and reword commits "\
"in the pull request to our preferred format.",
replacement: "`--no-autosquash` to opt out"
switch "--no-autosquash",
description: "Skip automatically reformatting and rewording commits in the pull request "\
"to the preferred format, even if supported on the target tap."
Expand Down
4 changes: 0 additions & 4 deletions Library/Homebrew/dev-cmd/pr-pull.rb
Original file line number Diff line number Diff line change
Expand Up @@ -32,10 +32,6 @@ def pr_pull_args
switch "--keep-old",
description: "If the formula specifies a rebuild version, " \
"attempt to preserve its value in the generated DSL."
switch "--autosquash",
description: "Automatically reformat and reword commits in the pull request to our "\
"preferred format.",
replacement: "`--no-autosquash` to opt out"
switch "--no-autosquash",
description: "Skip automatically reformatting and rewording commits in the pull request to our "\
"preferred format."
Expand Down
2 changes: 1 addition & 1 deletion Library/Homebrew/dev-cmd/pr-upload.rb
Original file line number Diff line number Diff line change
Expand Up @@ -90,7 +90,7 @@ def bottles_hash_from_json_files(json_files, args)
def pr_upload
args = pr_upload_args.parse

# odeprecated "`brew pr-upload --github-org`", "`brew pr-upload` without `--github-org`" if args.github_org
odeprecated "`brew pr-upload --github-org`", "`brew pr-upload` without `--github-org`" if args.github_org

json_files = Dir["*.bottle.json"]
odie "No bottle JSON files found in the current working directory" if json_files.blank?
Expand Down
2 changes: 1 addition & 1 deletion Library/Homebrew/formula_support.rb
Original file line number Diff line number Diff line change
Expand Up @@ -82,7 +82,7 @@ class BottleDisableReason
def initialize(type, reason)
@type = type
@reason = reason
odeprecated "bottle :#{@type}" if valid?
odisabled "bottle :#{@type}" if valid?
end

def unneeded?
Expand Down