Skip to content

Conversation

@Mick235711
Copy link
Contributor

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

Old-style Quick Look plugins (those with a .qlgenerator extension) were deprecated since Catalina, and Apple decided to remove support for them last year since Sequoia. This means that the 38 casks with qlplugin artifacts will no longer work on Sequoia and newer systems. Given that Sonoma support will likely be dropped from the CI next year, this means that shortly after macOS 27's release, these casks will no longer work on any Tier 1 configuration.

Most of the casks mentioned are not updated for several years at this point. Newer QL plugins will utilize the Quick Look Thumbnailing Extension and appear directly as a normal .app, for example QLVideo.

Therefore, this PR tries to add a reason in preparation for their deprecation.

@p-linnane
Copy link
Member

Thanks for the PR. I agree we should deprecate these, but I don't think we should add a new reason for a one-off. @Mick235711 Can you open a PR to deprecate these, but use the no_longer_meets_criteria? Thanks!

@p-linnane p-linnane closed this Sep 21, 2025
@bevanjkay
Copy link
Member

I hadn't had a chance to get to this yet, but I had the same thought as @p-linnane.
It would be good to update the documentation to either remove the references to the qlplugin artifact, or note that they are now deprecated.

@Mick235711
Copy link
Contributor Author

Thanks for the PR. I agree we should deprecate these, but I don't think we should add a new reason for a one-off. @Mick235711 Can you open a PR to deprecate these, but use the no_longer_meets_criteria? Thanks!

Sure, done.

It would be good to update the documentation to either remove the references to the qlplugin artifact, or note that they are now deprecated.

Sure, I opened another PR (#20744) to try to achieve that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants