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

[FileProvider] Update for Xcode14 binding #15790

Merged
merged 12 commits into from
Aug 31, 2022
4 changes: 3 additions & 1 deletion src/fileprovider.cs
Original file line number Diff line number Diff line change
Expand Up @@ -817,8 +817,10 @@ interface NSFileProviderPendingSetEnumerator : NSFileProviderEnumerator {
[Export ("refreshInterval")]
double RefreshInterval { get; }

[NoWatch, NoTV, NoMacCatalyst, Mac (13, 0), iOS (16, 0)]
#if XAMCORE_5_0
[Abstract]
#endif
[NoWatch, NoTV, NoMacCatalyst, Mac (13, 0), iOS (16, 0)]
[Export ("maximumSizeReached")]
bool MaximumSizeReached { [Bind ("isMaximumSizeReached")] get; }
}
Expand Down
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
# New abstract protocol member was added in Xcode14
# which is a breaking change so adding it to XAMCORE_5_0
!incorrect-protocol-member! NSFileProviderPendingSetEnumerator::isMaximumSizeReached is REQUIRED and should be abstract
Copy link
Member

Choose a reason for hiding this comment

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

Missing eol at the end

dalexsoto marked this conversation as resolved.
Show resolved Hide resolved
3 changes: 3 additions & 0 deletions tests/xtro-sharpie/common-FileProvider.ignore
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
# New abstract protocol member was added in Xcode14
# which is a breaking change so adding it to XAMCORE_5_0
!incorrect-protocol-member! NSFileProviderPendingSetEnumerator::isMaximumSizeReached is REQUIRED and should be abstract
Copy link
Member

Choose a reason for hiding this comment

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

Same

dalexsoto marked this conversation as resolved.
Show resolved Hide resolved