Install size

[tyuwags]

• Have you followed the guidelines in our Contributing document?
• Have you checked to ensure there aren't other open Pull Requests for the same change?
• Have you added an explanation of what your changes do and why you'd like us to include them?
• Have you written new tests for your changes? Here's an example.
• Have you successfully run brew style with your changes locally?
• Have you successfully run brew typecheck with your changes locally?
• Have you successfully run brew tests with your changes locally?

---

resolve #18373

As discussed in the issue, these changes add the --ask option and the HOMEBREW_ASK environment variable to display the packages and dependencies that will be installed, along with their sizes, and prompt the user for input.

[MikeMcQuaid]

Thanks @tyuwags, great work so far!

[MikeMcQuaid]

Great work so far, I think we're almost there!

[MikeMcQuaid]

Suggested change

	puts "#{::Utils.pluralize("Formul", sized_formulae.count, plural: "ae",
	singular: "a")} \
	(#{sized_formulae.count}): #{sized_formulae.join(", ")}\n\n"
	puts "#{::Utils.pluralize("Formula")} \
	(#{sized_formulae.count}): #{sized_formulae.join(", ")}\n\n"

should work

[tyuwags]

it's not working, its adding a s to formula, should be ae

[MikeMcQuaid]

Hmm, there's other usage of Utils.pluralize("Formula"), can you see if they are working/broken?

[tyuwags]

I cant see other usage of Utils.pluralize("Formula")
here the only method in Utils

def self.pluralize(stem, count, plural: "s", singular: "", include_count: false)
    prefix = include_count ? "#{count} " : ""
    suffix = (count == 1) ? singular : plural
    "#{prefix}#{stem}#{suffix}"
  end