Deprecate --new-formula/--new-cask options.

[reitermarkus]

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

---

See Homebrew/homebrew-cask#161724.

Both --new-formula as well as --new-cask do the same thing as --new, so they're not needed anymore.

[MikeMcQuaid]

Thanks @reitermarkus, looking good so far. A few suggestions so we can get this deprecated sooner rather than later.

[MikeMcQuaid]

Suggested change

	new_cask = args.new? || args.new_cask?
	new_formula = args.new? || args.new_formula?
	new_cask = args.new?
	if args.new_cask?
	# odeprecated "brew audit --new-cask"
	new_cask = true
	end
	new_formula = args.new?
	if args.new_formula?
	# odeprecated "brew audit --new-formula"
	new_formula = true
	end

[MikeMcQuaid]

@reitermarkus Still think this would be good to do so we delete and print deprecation messages on usage.

[reitermarkus]

odeprecated is automatically done by CLI::Parser once the replacements are uncommented on the switch calls.

[MikeMcQuaid]

Yup but we're reliant on the commenting and changing of odeprecated to odisabled process to get this code removed.

[reitermarkus]

I already added a odeprecated comment above on the switch calls. Once the switch calls use disable: true, calling args.new_formula?/args.new_cask? here will fail CI anyways.

[reitermarkus]

Also added a comment and disable: false to make it greppable.

[MikeMcQuaid]

Thanks again @reitermarkus!