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

Wireless charger #209

Merged
merged 10 commits into from
Oct 5, 2019
Merged

Conversation

gianpispi
Copy link
Contributor

@gianpispi gianpispi commented Oct 1, 2019

Added wireless charging capability variable

Closes #201

@devicekit-danger-bot
Copy link

devicekit-danger-bot commented Oct 1, 2019

1 Error
🚫 Please rebase to get rid of the merge commits in this PR
3 Warnings
⚠️ Plist changed, don’t forget to localize your plist values
⚠️ Source/Device.generated.swift#L667 - Prefer non-optional booleans over optional booleans.
⚠️ Source/Device.generated.swift#L1262 - Prefer empty collection over optional collection.

SwiftLint found issues

Warnings

File Line Reason
Device.generated.swift 667 Prefer non-optional booleans over optional booleans.
Device.generated.swift 1262 Prefer empty collection over optional collection.

Generated by 🚫 Danger

Copy link
Member

@Zandor300 Zandor300 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You didn't actually add booleans for all devices for your new supportsWirelessCharging property? Can't see it in your file diff...

@gianpispi
Copy link
Contributor Author

Sorry @Zandor300, what do you mean by that?

@Zandor300
Copy link
Member

@gianpispi You added a property to the initializer of the gyb Device class. Now you also need to add this boolean for every device that is in there so that the allDevicesWithWirelessChargingSupport and supportsWirelessCharging functions will be auto-generated from that.

@gianpispi
Copy link
Contributor Author

Okay got it! Now I think it's all right. Thanks for helping ;)

Source/Device.swift.gyb Outdated Show resolved Hide resolved
Source/Device.swift.gyb Outdated Show resolved Hide resolved
Source/Device.generated.swift Outdated Show resolved Hide resolved
Source/Device.swift.gyb Show resolved Hide resolved
Copy link
Collaborator

@denisenepraunig denisenepraunig left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @gianpispi - really cool feature! Thanks @Zandor300 for the review!

@denisenepraunig denisenepraunig merged commit 81ca67d into devicekit:master Oct 5, 2019
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.

Feature Request: Check if device supports Wireless Charger
5 participants