Skip to content

Rename iPad Mini to iPad mini for Apple naming consistency#478

Merged
Zandor300 merged 3 commits intomasterfrom
copilot/fix-ipad-mini-naming
Mar 2, 2026
Merged

Rename iPad Mini to iPad mini for Apple naming consistency#478
Zandor300 merged 3 commits intomasterfrom
copilot/fix-ipad-mini-naming

Conversation

Copy link
Contributor

Copilot AI commented Dec 3, 2025

  • Update Source/Device.swift.gyb to change "iPad Mini" to "iPad mini" in strings and comments (lines 120-126)
  • Regenerate Source/Device.generated.swift using the gyb tool
  • Update Tests/Tests.swift to match new "iPad mini" string values (lines 358-362)
  • Update CHANGELOG.md to reflect "iPad mini" naming (4 occurrences)
  • Add changelog entry under Unreleased for iPad mini naming fix
  • Run code review
  • Run CodeQL security scan
Original prompt

This section details on the original issue you should resolve

<issue_title>iPad Mini vs. iPad mini</issue_title>
<issue_description>Thank you for creating this amazing library.

If anyone is excited by nitpicky details, for consistency with Apple's naming, the capitalization of "iPad Mini" should be "iPad mini".

See Device.swift.gyb, lines 120 through 126

Device("iPadMini", "Device is an [iPad Mini](https://support.apple.com/kb/SP661)", "https://support.apple.com/library/APPLE/APPLECARE_ALLGEOS/SP661/sp661_ipad_mini_color.jpg", ["iPad2,5", "iPad2,6", "iPad2,7"], 7.9, (3, 4), "iPad Mini", "iPad Mini", 163, False, True, False, False, False, False, False, False, False, False, 0, False, 1, False, "a5", False, False),

Please let me know if you'd like me to create a PR for this issue.</issue_description>

<agent_instructions>Rename all iPad Mini to iPad mini with lowercase m. Only inside strings and comments. Keep the enum still with capital M according to lowerCamelCasing.</agent_instructions>

Comments on the Issue (you are @copilot in this section)


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

iPad Mini vs. iPad mini

2 participants