-
-
Notifications
You must be signed in to change notification settings - Fork 10.8k
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
Do not require a DSL version in cask header #15901
Conversation
Following up on #15782. Casks can now be created with no DSL version in the header. For example: cask 'no-dsl-version' do version :latest ... end Casks with a DSL version in the header are still valid, but the DSL version will be ignored.
@caskroom/maintainers Any comments? If not, I'll merge at 10pm UTC. |
Why not make it a symbol as well ( |
@vitorgalvao We could, though I'm not sure what the benefit would be. |
Also, symbols can't contain hyphens unless surrounded in quotes. So it would need to be either |
Ah, no, ignore it. I was thinking of having it as symbol for consistency with what we had, but just realised I misinterpreted it. On first look it seemed like we needed to be explicit about not having a version, but I see now it stands in to be the name of the cask. |
LGTM. After this merge, existing Casks could be changed to the new header as time permits. That would also be a (silent) indicator that a maintainer has looked at the Cask and ensured that everything checks out. That would also be a good time to implement any other breaking changes, so it might be best to avoid a mass update (proactively) until the core stabilizes a bit more/the backlog of current PRs that deal with core functionality are merged or otherwise addressed. |
To be honest, I'm not too jazzed about combing through ~3000 casks to find bugs. A mass update would be as easy as Any PRs that include truly breaking changes should also include fixes for affected casks (in separate commits). |
Yeah, on second thought, maybe not the best idea. I forgot how many Casks we had! I would say then that this PR doesn't seem to have any regressions and so a merge, followed by a mass scripted update whenever convenient should work out well. |
Do not require a DSL version in cask header
Following up on #15782. Casks can now be created with no DSL version in the header. For example:
Casks with DSL versions in the header are still valid, but the DSL version will be ignored.