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

Convert if MacOS.version blocks to on_system blocks #137512

Closed
p-linnane opened this issue Dec 12, 2022 · 5 comments
Closed

Convert if MacOS.version blocks to on_system blocks #137512

p-linnane opened this issue Dec 12, 2022 · 5 comments
Labels

Comments

@p-linnane
Copy link
Member

p-linnane commented Dec 12, 2022

Description of issue

This is a tracking issue to monitor the progress of converting all if MacOS.version blocks into the new on_system blocks. We have better coverage with brew readall using these new blocks.

@Rylan12
Copy link
Member

Rylan12 commented Dec 28, 2022

Thanks, everyone, for helping out with this!

I'll have a PR in homebrew/brew soon to enforce the on_system blocks

@bevanjkay
Copy link
Member

@Rylan12 There's a few more in our other Homebrew-cask-* repos that might need attention, there's a couple of complicated ones in the drivers and versions repos.

@Rylan12
Copy link
Member

Rylan12 commented Dec 28, 2022

I'm going to push forward, for now, to allow HOMEBREW_INSTALL_FROM_API to move forward (API installs aren't available for those repos right now, anyway) but I'll make sure to add a todo to hit all of these.

It looks like Homebrew/cask-drivers has 64 violations and Homebrew/cask-versions has 18.

@Rylan12
Copy link
Member

Rylan12 commented Dec 28, 2022

Hmm, on second thought it might actually take more time to get the audit to be main-tap-specific so I might just start running through these. not sure yet

@bevanjkay
Copy link
Member

bevanjkay commented Dec 28, 2022

@Rylan12 We can hit these next, especially if it's blocking the API project.

EDIT: I've opened PRs to update all the ones in versions.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jan 28, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

No branches or pull requests

4 participants