Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

[clockkit] Add missing 'CLKComplicationFamily' enum values#4818

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged

Conversation

@VincentDondain
Copy link
Contributor

From Xcode 10 GM API diff:https://github.com/xamarin/xamarin-macios/wiki/ClockKit-watchOS-xcode10-GM

--- /Applications/Xcode10-beta6.app/Contents/Developer/Platforms/WatchOS.platform/Developer/SDKs/WatchOS.sdk/System/Library/Frameworks/ClockKit.framework/Headers/CLKDefines.h2018-08-10 17:39:53.000000000 -0400+++ /Applications/Xcode10GM.app/Contents/Developer/Platforms/WatchOS.platform/Developer/SDKs/WatchOS.sdk/System/Library/Frameworks/ClockKit.framework/Headers/CLKDefines.h2018-08-29 05:55:06.000000000 -0400@@ -19,6 +19,10 @@     CLKComplicationFamilyCircularSmall                                                        = 4,     CLKComplicationFamilyExtraLarge CLK_AVAILABLE_WATCHOS_IOS(3_0, 10_0)                      = 7,+    CLKComplicationFamilyGraphicCorner CLK_AVAILABLE_WATCHOS_IOS(5_0, 12_0)                 = 8,+    CLKComplicationFamilyGraphicBezel CLK_AVAILABLE_WATCHOS_IOS(5_0, 12_0)                  = 9,+    CLKComplicationFamilyGraphicCircular CLK_AVAILABLE_WATCHOS_IOS(5_0, 12_0)               = 10,+    CLKComplicationFamilyGraphicRectangular CLK_AVAILABLE_WATCHOS_IOS(5_0, 12_0)            = 11, };

These are important enum values related to the new "graphic" complications types.

From Xcode 10 GM API diff:https://github.com/xamarin/xamarin-macios/wiki/ClockKit-watchOS-xcode10-GM``` diff--- /Applications/Xcode10-beta6.app/Contents/Developer/Platforms/WatchOS.platform/Developer/SDKs/WatchOS.sdk/System/Library/Frameworks/ClockKit.framework/Headers/CLKDefines.h2018-08-10 17:39:53.000000000 -0400+++ /Applications/Xcode10GM.app/Contents/Developer/Platforms/WatchOS.platform/Developer/SDKs/WatchOS.sdk/System/Library/Frameworks/ClockKit.framework/Headers/CLKDefines.h2018-08-29 05:55:06.000000000 -0400@@ -19,6 +19,10 @@     CLKComplicationFamilyCircularSmall                                                        = 4,     CLKComplicationFamilyExtraLarge CLK_AVAILABLE_WATCHOS_IOS(3_0, 10_0)                      = 7,     +    CLKComplicationFamilyGraphicCorner CLK_AVAILABLE_WATCHOS_IOS(5_0, 12_0)                 = 8,+    CLKComplicationFamilyGraphicBezel CLK_AVAILABLE_WATCHOS_IOS(5_0, 12_0)                  = 9,+    CLKComplicationFamilyGraphicCircular CLK_AVAILABLE_WATCHOS_IOS(5_0, 12_0)               = 10,+    CLKComplicationFamilyGraphicRectangular CLK_AVAILABLE_WATCHOS_IOS(5_0, 12_0)            = 11, };```These are important enum values related to the new "graphic" complications types.
Copy link
Contributor

@spouliotspouliot left a comment

Choose a reason for hiding this comment

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

minor: ordering is automatic, you do not need them unless they are not continuous

@monojenkins
Copy link
Collaborator

Build failure
Build was aborted

@monojenkins
Copy link
Collaborator

Build failure
Build succeeded
API Diff (from stable)
API Diff (from PR only) (please review changes)
Generator Diff (only version changes)
🔥Test run failed 🔥

Test results

1 tests failed, 0 tests skipped, 79 tests passed.

Failed tests

  • link all/Mac/Debug: Failed (Test run failed.)

@VincentDondain
Copy link
ContributorAuthor

Known test failure:https://github.com/xamarin/maccore/issues/649

@VincentDondainVincentDondain merged commita4268f2 intoxcode10Sep 14, 2018
@VincentDondainVincentDondain deleted the xcode10-clockkit-gm-missing-enum-values branchSeptember 14, 2018 22:45
@t9mike
Copy link

Rookie question: when will this update be in one of the release channels? I need new watchOS 5 complications.

@t9mike
Copy link

Rookie question: when will this update be in one of the release channels? I need new watchOS 5 complications.

So Xcode Preview 10 channel is behind stable now. OK, I see it new watchOS types now that I switched to Stable.

Would be really helpful if a modal dialog warning was shown on VS.NET Mac launch or if manually check for new version if channel you are on is BEHIND a "higher" channel.

@t9mike
Copy link

Would be really helpful if a modal dialog warning was shown on VS.NET Mac launch or if manually check for new version if channel you are on is BEHIND a "higher" channel.

Seehttps://developercommunity.visualstudio.com/content/problem/340745/check-for-updates-needs-to-provide-status-of-behin.html andhttps://developercommunity.visualstudio.com/content/problem/340746/proactively-warn-user-if-they-are-on-update-channe.html.

@VincentDondain
Copy link
ContributorAuthor

Hi@t9mike sorry for not answering earlier I totally missed your responses. My bad I apologize.

Thanks for filing those issues against Visual Studio for Mac, your feedback seems valid to me, the team will investigate.

Also it's a bit late now because you have the new enum values but in the future if we're ever missing enum values (we shouldn't!) know that you can just use the new values directly.

So in anif statement inGetCurrentTimelineEntry for instance:complication.Family == CLKComplicationFamily.GraphicCorner is equivalent tocomplication.Family == (CLKComplicationFamily)8 (;

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

Reviewers

@rolfbjarnerolfbjarnerolfbjarne approved these changes

+1 more reviewer

@spouliotspouliotspouliot approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@VincentDondain@monojenkins@t9mike@rolfbjarne@spouliot@xamarin-release-manager

[8]ページ先頭

©2009-2025 Movatter.jp