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

[maccatalyst] Increase iOS compatibility by removing XAMCORE_3_0 #11238

Merged
merged 2 commits into from
Apr 20, 2021

Conversation

spouliot
Copy link
Contributor

XAMCORE_3_0 was added in tvOS and watchOS where binary compatibility
was not a consideration. The work to update source code for XAMCORE_3_0
was also minimal.

The former is not the always the case for MacCatalyst, while the latter
is still true.

Fix #11237 and #11236

`XAMCORE_3_0` was added in tvOS and watchOS where binary compatibility
was not a consideration. The work to update source code for `XAMCORE_3_0`
was also minimal.

The former is not the always the case for MacCatalyst, while the latter
is still true.

Fix dotnet#11237 and dotnet#11236
@spouliot spouliot added the not-notes-worthy Ignore for release notes label Apr 19, 2021
@vs-mobiletools-engineering-service2
Copy link
Collaborator

❌ Tests failed on Build ❌

Tests failed on Build.

API diff

✅ API Diff from stable

View API diff

Test results

2 tests failed, 97 tests passed.

Failed tests

  • introspection/Mac Catalyst/Debug: Failed (Tests run: 42 Passed: 38 Inconclusive: 0 Failed: 1 Ignored: 3)
  • introspection/watchOS 32-bits - simulator/Debug (watchOS 5.0): LaunchFailure

Pipeline on Agent XAMBOT-1104.BigSur
Merge 3d55b23 into dec81b6

@vs-mobiletools-engineering-service2
Copy link
Collaborator

🔥 Tests failed catastrophically on Build (no summary found). 🔥

Result file $(TEST_SUMMARY_PATH) not found.

Pipeline on Agent
Merge 4b5ea8c into dec81b6

@spouliot
Copy link
Contributor Author

Invoke-RestMethod : Unable to read data from the transport connection: An existing connection was forcibly closed by the remote host..
:(

@spouliot
Copy link
Contributor Author

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@vs-mobiletools-engineering-service2
Copy link
Collaborator

❌ Tests failed on Build ❌

Tests failed on Build.

API diff

✅ API Diff from stable

View API diff

Test results

1 tests failed, 98 tests passed.

Failed tests

  • introspection/watchOS 32-bits - simulator/Debug (watchOS 5.0): LaunchFailure

Pipeline on Agent XAMBOT-1104.BigSur
Merge 4b5ea8c into dec81b6

@spouliot
Copy link
Contributor Author

Failure is unrelated, known issue -> https://github.com/xamarin/maccore/issues/2411

@spouliot spouliot merged commit 7d2b389 into dotnet:main Apr 20, 2021
@spouliot spouliot deleted the maccat-not-xamcore3 branch April 20, 2021 12:59
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.

[maccatalyst] UIKeyboard.BoundsUserInfoKey member is missing
5 participants