formula: allow excluding deprecate! reason when disable! exists

[cho-m]

• 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?

---

[cho-m]

For convenience, I've set @deprecation_reason even if before @deprecation_date. I don't think this should be an issue and information can be used on formulae page or by 3rd-party usage of API.

Otherwise, would have to deal with resetting the info to nil.

[p-linnane]

This is a fantastic idea. Thanks @cho-m!

[reitermarkus]

Not really relevant for this PR, but are these automatically sorted in this order by another cop? To me it would make more sense to have deprecate! sorted first.

[cho-m]

It is from AST order:

brew/Library/Homebrew/ast_constants.rb

Lines 27 to 28 in 1d19230

	[{ name: :disable!, type: :method_call }],
	[{ name: :deprecate!, type: :method_call }],

Could reverse them if we wanted, though would need to make sure about handling. I did try to add code so order doesn't matter.

[cho-m]

Will wait until after #18733 is merged to finish this up.