Skip to content

[uikit] Update for Xcode 9 GM - #2670

Merged
spouliot merged 2 commits into
dotnet:xcode9from
VincentDondain:uikit-gm
Sep 13, 2017
Merged

[uikit] Update for Xcode 9 GM#2670
spouliot merged 2 commits into
dotnet:xcode9from
VincentDondain:uikit-gm

Conversation

@VincentDondain

Copy link
Copy Markdown
Contributor

No description provided.

Comment thread src/uikit.cs Outdated
[Export ("bounds")]
CGRect Bounds { get; }

[Watch (4,0), TV (11,0), iOS (11,0)]

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

is that type included in watchOS ?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

not it's not in watchOS headers, let's remove it to avoid confusion

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yea and actually all that code is under a #if !WATCH I didn't see...

Comment thread src/uikit.cs Outdated
[Export ("bounds")]
CGRect Bounds { get; }

[Watch (4,0), TV (11,0), iOS (11,0)]

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

not it's not in watchOS headers, let's remove it to avoid confusion

@monojenkins

Copy link
Copy Markdown
Contributor

Build failure

@monojenkins

Copy link
Copy Markdown
Contributor

Build failure

@rolfbjarne

Copy link
Copy Markdown
Member

Test failures are unrelated (xammac password failure - bug #58394).

@spouliot
spouliot merged commit 7c151e1 into dotnet:xcode9 Sep 13, 2017
@VincentDondain
VincentDondain deleted the uikit-gm branch September 13, 2017 18:42
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.

6 participants