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

[IntentsUI] Update bindings for Xcode13.0 betas 1, 4 #13056

Merged
merged 2 commits into from
Oct 22, 2021

Conversation

rachelkang
Copy link
Contributor

Update bindings for Xcode13.0 betas 1, 4 (No diffs for betas 2,3,5,6)
Note that IntentsUI previously was not compiled on either Mac or MacCatalyst

@rachelkang rachelkang added the note-highlight Worth calling out specifically in release notes label Oct 20, 2021
@rachelkang rachelkang added this to the xcode13.0 milestone Oct 20, 2021
src/frameworks.sources Outdated Show resolved Hide resolved
src/intentsui.cs Outdated Show resolved Hide resolved
@vs-mobiletools-engineering-service2
Copy link
Collaborator

❌ [PR Build] Tests failed on Build ❌

Tests failed on Build.

API diff

✅ API Diff from stable

View API diff

GitHub pages

Results can be found in the following github pages (it might take some time to publish):

Test results

1 tests failed, 101 tests passed.

Failed tests

  • link sdk/Mac Catalyst [dotnet]/Debug [dotnet]: Failed (Test run crashed (exit code: 134).
    Tests run: 129 Passed: 114 Inconclusive: 6 Failed: 0 Ignored: 15)

Pipeline on Agent XAMBOT-1094.BigSur'
Merge 816766b into 1cff1dd

Copy link
Member

@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.

@rachelkang looks good, I'm just wondering about the mac catalyst version.

@@ -110,13 +125,14 @@ interface INUIAddVoiceShortcutViewController {
[NullAllowed, Export ("delegate", ArgumentSemantic.Weak)]
NSObject WeakDelegate { get; set; }

[NoMacCatalyst]
[MacCatalyst (13,4)]
Copy link
Member

Choose a reason for hiding this comment

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

13,4?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@@ -143,13 +164,14 @@ interface INUIEditVoiceShortcutViewController {
[NullAllowed, Export ("delegate", ArgumentSemantic.Weak)]
NSObject WeakDelegate { get; set; }

[NoMacCatalyst]
[MacCatalyst (13,4)]
Copy link
Member

Choose a reason for hiding this comment

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

13,4?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Comment on lines +113 to +117
#if MONOMAC
[BaseType (typeof (NSViewController))]
#else
[BaseType (typeof (UIViewController))]
#endif
Copy link
Member

Choose a reason for hiding this comment

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

This could have done with a using, but I'm ok with it.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

For my future reference - what would be the alternative?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

nevermind, I misread! cool :)

@rachelkang
Copy link
Contributor Author

test failure unrelated: https://github.com/xamarin/maccore/issues/2494

@rachelkang rachelkang merged commit 410237f into xamarin:main Oct 22, 2021
@mandel-macaque
Copy link
Member

/sudo backport xcode13.1

@vs-mobiletools-engineering-service2
Copy link
Collaborator

Backport Job to branch **xcode13.1 ** Created! The magic is happening here

@vs-mobiletools-engineering-service2
Copy link
Collaborator

Oh no! Backport failed! Please see https://devdiv.visualstudio.com/DevDiv/_build/results?buildId=5395344 for more details.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
note-highlight Worth calling out specifically in release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants