fix deprecate! and disable! message

[Rylan12]

• 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 tests with your changes locally?

---

Follow up to #8530

Fixes an issue where the wrong message is displayed

Before this PR:

$ brew install tj
Error: tj has been disabled because it {:does_not_build=>"does not build", :no_license=>"has no license", :repo_archived=>"has an archived upstream repository", :repo_removed=>"has a removed upstream repository", :unmaintained=>"is not maintained upstream", :unsupported=>"is not supported upstream", :deprecated_upstream=>"is deprecated upstream", :versioned_formula=>"is a versioned formula"}!

After:

$ brew install tj
Error: tj has been disabled because it has no license!

CC: @SeekingMeaning

[reitermarkus]

Nitpick: Why is “it” not part of the reason? That way it would also read like a sentence inside of formulae.

[Rylan12]

Nitpick: Why is “it” not part of the reason? That way it would also read like a sentence inside of formulae.

It turned out that almost all descriptions started with it so it was added to the message by default to remove the need for typing it every time. See Homebrew/homebrew-core#60321 (comment)

With the new symbols, most reasons should be something like disable! because: :unmaintained or similar which removes the need for it at all. It's only an issue when you need to provide a custom description. There is also a style check for it so it's not accidentally added twice.

[MikeMcQuaid]

Thanks again @Rylan12!