Skip to content

Conversation

@ghost
Copy link

@ghost ghost commented Oct 28, 2019

Add AuthenticationServices bindings for xcode11.2

@ghost ghost requested a review from spouliot as a code owner October 28, 2019 14:58
Copy link
Contributor

@mandel-macaque mandel-macaque left a comment

Choose a reason for hiding this comment

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

Changing the enums value, is a breaking change like: #7292

Copy link
Contributor

@mandel-macaque mandel-macaque left a comment

Choose a reason for hiding this comment

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

I missed that Default == Sigin, should be ok.

Copy link
Contributor

@spouliot spouliot left a comment

Choose a reason for hiding this comment

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

The headers might say so but, in reality, there never was a tvOS 13.1. It went from 13.0 to 13.2

@monojenkins
Copy link
Collaborator

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

Test results

1 tests failed, 90 tests passed.

Failed tests

  • introspection/Mac Modern/Debug: Failed (Test run failed.)

@monojenkins
Copy link
Collaborator

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

Test results

5 tests failed, 86 tests passed.

Failed tests

  • introspection/Mac Modern/Debug: Failed (Test run failed.)
  • xammac tests/Mac Modern/Debug: Failed (Test run failed.)
  • xammac tests/Mac Modern/Release: Failed (Test run failed.)
  • xammac tests/Mac Modern/Release: Failed (Test run failed.)
  • monotouch-test/watchOS 32-bits - simulator/Debug: Crashed

@ghost ghost merged commit 25da8b4 into dotnet:xcode11.2 Oct 29, 2019
@rolfbjarne rolfbjarne added the not-notes-worthy Ignore for release notes label Sep 10, 2020
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

not-notes-worthy Ignore for release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants