Skip to content

Add property to determine if device is compatible with Apple Pencil - #178

Closed
SomeRandomiOSDev wants to merge 4 commits into
devicekit:masterfrom
SomeRandomiOSDev:master
Closed

Add property to determine if device is compatible with Apple Pencil#178
SomeRandomiOSDev wants to merge 4 commits into
devicekit:masterfrom
SomeRandomiOSDev:master

Conversation

@SomeRandomiOSDev

Copy link
Copy Markdown
Contributor

Added a property to the Device enum to determine is the given device can use the Apple Pencil and which version of the Apple Pencil is supported. Current list of supported devices taken off of the Apple Pencil support page: https://support.apple.com/en-us/HT205236

@devicekit-danger-bot

devicekit-danger-bot commented Apr 10, 2019

Copy link
Copy Markdown
1 Error
🚫 Please rebase to get rid of the merge commits in this PR
4 Warnings
⚠️ Plist changed, don’t forget to localize your plist values
⚠️ Source/Device.generated.swift#L618 - Prefer non-optional booleans over optional booleans.
⚠️ Source/Device.generated.swift#L620 - TODOs should be resolved (Longterm we need a better solu…).
⚠️ Source/Device.generated.swift#L1172 - Prefer empty collection over optional collection.

SwiftLint found issues

Warnings

File Line Reason
Device.generated.swift 618 Prefer non-optional booleans over optional booleans.
Device.generated.swift 1172 Prefer empty collection over optional collection.
Device.generated.swift 620 TODOs should be resolved (Longterm we need a better solu...).

Generated by 🚫 Danger

# Conflicts:
#	Source/Device.generated.swift
#	Source/Device.swift.gyb
@denisenepraunig

Copy link
Copy Markdown
Collaborator

Hello @SomeRandomiOSDev! Could you please rebase your changes onto master - I have merged some breaking changes yesterday and there seem to be merge conflicts. Then I will review the changes, but so far so good and @Zandor300 already looked at the PR in the previous branch and it looked good to him. I will then merge this PR.

@SomeRandomiOSDev

Copy link
Copy Markdown
Contributor Author

Rebasing isn't working out so well due to conflicts between the master branch and these changes. I'm closing this and will open a new PR with my changes cherry picked after the latest changes on the master branch

@SomeRandomiOSDev
SomeRandomiOSDev deleted the master branch April 11, 2019 12:37
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.

3 participants