-
Notifications
You must be signed in to change notification settings - Fork 511
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
[ClockKit] Update to Xcode 9 Beta 1 #2208
Conversation
Build failure |
src/clockkit.cs
Outdated
@@ -53,16 +53,20 @@ interface CLKComplicationDataSource { | |||
[Export ("getTimelineEntriesForComplication:afterDate:limit:withHandler:")] | |||
void GetTimelineEntriesAfterDate (CLKComplication complication, NSDate afterDate, nuint limit, Action<CLKComplicationTimelineEntry []> handler); | |||
|
|||
[Deprecated (PlatformName.WatchOS, 4, 0, message: "Use WKRefreshBackgroundTask instead")] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please take a look at https://github.com/xamarin/xamarin-macios/wiki/BINDINGS#availability-attributes-messages as a refresher (:
Your message should be Use 'WKRefreshBackgroundTask' instead.
src/clockkit.cs
Outdated
[Export ("getNextRequestedUpdateDateWithHandler:")] | ||
void GetNextRequestedUpdateDate (Action<NSDate> handler); | ||
|
||
[Deprecated (PlatformName.WatchOS, 4, 0, message: "Use `WKRefreshBackgroundTask` instead")] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
https://github.com/xamarin/xamarin-macios/wiki/BINDINGS#rule-2 please use apostrophes not grave accent (and add a dot at the end) (:
src/clockkit.cs
Outdated
[Export ("requestedUpdateDidBegin")] | ||
void RequestedUpdateDidBegin (); | ||
|
||
[Deprecated (PlatformName.WatchOS, 4, 0, message: "Use `WKRefreshBackgroundTask` instead")] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same
src/clockkit.cs
Outdated
[Export ("requestedUpdateBudgetExhausted")] | ||
void RequestedUpdateBudgetExhausted (); | ||
|
||
// this was @required in watchOS 2.x but is now deprecated and downgraded to @optional in watchOS 3 (betas) | ||
[Deprecated (PlatformName.WatchOS, 3, 0, message: "Use `GetLocalizableSampleTemplate` instead")] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same
@VincentDondain Done! And thanks for the refresher! |
Build success |
New commits in xamarin/maccore: * xamarin/maccore@ef861db73e [devops] Enhance Xcode renaming script (xamarin#2208) Diff: https://github.com/xamarin/maccore/compare/92a06f73033c07fe97735c1fa10e2fa0a9222c79..ef861db73e7c0f74131d08d5afa64841d6b00e0b
) New commits in xamarin/maccore: * xamarin/maccore@ef861db73e [devops] Enhance Xcode renaming script (#2208) Diff: https://github.com/xamarin/maccore/compare/92a06f73033c07fe97735c1fa10e2fa0a9222c79..ef861db73e7c0f74131d08d5afa64841d6b00e0b
No description provided.