Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

requirements: define cask on base class. #4090

Merged
merged 1 commit into from
Apr 18, 2018
Merged

requirements: define cask on base class. #4090

merged 1 commit into from
Apr 18, 2018

Conversation

MikeMcQuaid
Copy link
Member

The cask attribute doesn't make as much sense on Linux but can be ignored there. The advantage of this change is that (like #4086) it allows figuring out the relevant cask for a formulae requirement on a Linux machine.

The `cask` attribute doesn't make as much sense on Linux but can be
ignored there. The advantage of this change is that (like #4086) it
allows figuring out the relevant cask for a formulae requirement on a
Linux machine.
@MikeMcQuaid MikeMcQuaid added the in progress Maintainers are working on this label Apr 18, 2018
@MikeMcQuaid MikeMcQuaid merged commit dcf2068 into Homebrew:master Apr 18, 2018
@MikeMcQuaid MikeMcQuaid deleted the cask-generic-requirements branch April 18, 2018 09:08
@MikeMcQuaid MikeMcQuaid removed the in progress Maintainers are working on this label Apr 18, 2018
@lock lock bot added the outdated PR was locked due to age label May 18, 2018
@lock lock bot locked as resolved and limited conversation to collaborators May 18, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
outdated PR was locked due to age
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant