Skip to content
Snippets Groups Projects
Commit e686a392 authored by Rylan Polster's avatar Rylan Polster
Browse files

bump-formula-pr: add message to check resources

parent 227da3aa
No related branches found
No related tags found
No related merge requests found
......@@ -330,11 +330,22 @@ module Homebrew
end
unless args.dry_run?
PyPI.update_python_resources! formula, new_formula_version, silent: args.quiet?, ignore_non_pypi_packages: true
resources_checked = PyPI.update_python_resources! formula, new_formula_version,
silent: args.quiet?, ignore_non_pypi_packages: true
end
run_audit(formula, alias_rename, old_contents, args: args)
pr_message = "Created with `brew bump-formula-pr`."
if resources_checked.nil? && formula.resources.present? &&
formula.resources.any? { |resource| resource.name != "homebrew-virtualenv" }
pr_message += <<~EOS
`resource` blocks may require updates.
EOS
end
pr_info = {
sourcefile_path: formula.path,
old_contents: old_contents,
......@@ -345,7 +356,7 @@ module Homebrew
previous_branch: previous_branch,
tap: formula.tap,
tap_full_name: tap_full_name,
pr_message: "Created with `brew bump-formula-pr`.",
pr_message: pr_message,
}
GitHub.create_bump_pr(pr_info, args: args)
end
......
......@@ -56,6 +56,7 @@ module PyPI
[json["info"]["name"], sdist["url"], sdist["digests"]["sha256"]]
end
# Return true if resources were checked (even if no change)
def update_python_resources!(formula, version = nil, print_only: false, silent: false,
ignore_non_pypi_packages: false)
......@@ -160,5 +161,7 @@ module PyPI
end
s.sub! inreplace_regex, new_resource_blocks
end
true
end
end
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment