Deprecate license

[vitorgalvao]

Refs #16692.

• Deprecate license from core.
• Remove license documentation.
• Remove license from all official taps.

[jawshooah]

Remove license tests.

I don't think this is necessary. As long as it's just deprecated, and not completely removed, there should be tests for it.

[vitorgalvao]

Why keep the tests, though? Ideally (or, as I see it), it being deprecated means using casks that use it won’t blow up in your face, but auditing and testing for it will. I.e., if you’re “simply a user”, everything will work as expected, but if you’re developing at all, you should be actively informed and discouraged about something deprecated.

[jawshooah]

if you’re developing at all, you should be actively informed and discouraged about something deprecated

That's fine, and usually how it works in software development when working with languages that support API deprecation as a feature (e.g. Java). But generally, or at least in my experience, the tests remain until the deprecated API is removed in the next major release.

I'm fine with audit producing a warning for casks which use deprecated APIs, but the tests should remain until support is completely removed from the core. This ensures that the feature still works, despite being deprecated (and keeps coverage metrics from degrading as well).

[vitorgalvao]

Sounds fair. Remove that point from the top post.

[adamo57]

what do you mean when you say, "Remove license from all official taps."?

[reitermarkus]

Every cask has a license stanza, which is, and never was, used for anything.

"Remove license from all official taps." means removing the license stanza from all casks in the official taps (currently only two: caskroom/cask and caskroom/versions).

[adamo57]

So is this supposed to be a PR for both caskroom/cask and caskroom/versions repos? I'm having a little difficulty understanding how this is supposed to work under the same branch.

[reitermarkus]

These are basically three separate tasks that will have to be completed in this order:

1. Deprecate license in core code, which is now located in Homebrew/brew.
2. Remove license from documentation, here in this repo.
3. Remove license from all casks, here in this repo and caskroom/versions.

[vitorgalvao]

There’s also casroom/eid and caskroom/fonts (the latter being the causer of the license stanza).

Removing stanzas is easily done with a simple script, though. Deprecating license from the core is what really matters.

[adidalal]

cask style needs to be updated, also, to not add in an empty blank line

[vitorgalvao]

@adidalal Already on it: https://github.com/caskroom/rubocop-cask/commit/05089851bee1366990bfd457f75935e4536c40fb and https://github.com/caskroom/rubocop-cask/commit/c7556c92cf729abd6d60b5fa162b44a3b6f81ed3.

[vitorgalvao]

Done! Thank you, @abookyun.