Add DeviceKit-style group arrays and installation docs#8
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 52ca7ca02e
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| static var allProDevices: [Device] { all.filter { $0.has(.pro) } } | ||
|
|
||
| /// All devices marked as mini. | ||
| static var allMiniDevices: [Device] { all.filter { $0.has(.mini) } } |
There was a problem hiding this comment.
Restrict mini group to iPad minis
For DeviceKit migration compatibility, allMiniDevices should not be every device carrying the generic .mini capability. The existing DeviceKit conversion maps mini form factor as device.is(.mini) && idiom == .pad and explicitly notes that HomePod mini and iPhone mini do not count (Development/Conversions/DeviceKitConversion.swift:542); this implementation also includes iPhone 12/13 mini, HomePod mini, and Mac mini, so callers porting iPad-mini-specific layout checks will start matching unrelated products.
Useful? React with 👍 / 👎.
Summary
Sources/DeviceKitGroups.swiftwith public DeviceKit-style group arrays such asallPhones,allPads,allWatches, and derived capability groups.INSTALLATION.mdwith SPM, manual installation, CocoaPods status, and Carthage status.AUDIT-2026-06-28.mddocumenting current coverage for iPhone 17/Air/17e, M5/M4 iPads, and current Apple Watch models.Notes
kudit/Devicedoes not model simulators as separate enum cases the way DeviceKit does.Testing