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

[CoreData] Update framework for Xcode 12 beta 4. #9405

Merged
merged 3 commits into from
Aug 20, 2020

Conversation

mandel-macaque
Copy link
Member

No description provided.

src/coredata.cs Outdated
[Watch (7,0), TV (14,0), Mac (11,0), iOS (14,0)]
[Static]
[Export ("batchInsertRequestWithEntityName:dictionaryHandler:")]
NSBatchInsertRequest BatchInsertRequest (string entityName, NSBatchInsertRequestDictionaryHandler handler);
Copy link
Contributor

Choose a reason for hiding this comment

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

no verb, Insert is not a verb here since it return a type of that name,
-> CreateBatchInsertRequest

src/coredata.cs Outdated
[Watch (7,0), TV (14,0), Mac (11,0), iOS (14,0)]
[Static]
[Export ("batchInsertRequestWithEntityName:managedObjectHandler:")]
NSBatchInsertRequest BatchInsertRequest (string entityName, NSBatchInsertRequestManagedObjectHandler handler);
Copy link
Contributor

Choose a reason for hiding this comment

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

same

src/coredata.cs Outdated

[Static]
[Export ("fetchEventsAfterDate:")]
NSPersistentCloudKitContainerEventRequest FetchEvents (NSDate date);
Copy link
Contributor

Choose a reason for hiding this comment

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

Minimally the parameter name should be afterDate, otherwise that information is lost

but I think it might be better to have FetchEventsAfter (NSDate date) since ...beforeDate: could be added later and cause confusion

src/coredata.cs Outdated

[Static]
[Export ("fetchEventsAfterEvent:")]
NSPersistentCloudKitContainerEventRequest FetchEvents ([NullAllowed] NSPersistentCloudKitContainerEvent @event);
Copy link
Contributor

Choose a reason for hiding this comment

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

After must be somewhere

Copy link
Member Author

Choose a reason for hiding this comment

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

Is after the next commit

@monojenkins
Copy link
Collaborator

Build failure
Build failed or was aborted

Build succeeded
API Diff (from stable)
ℹ️ API Diff (from PR only) (please review changes)
ℹ️ Generator Diff (please review changes)

src/CoreData/Enums.cs Show resolved Hide resolved
Co-authored-by: Whitney Schmidt <whschm@microsoft.com>
@monojenkins
Copy link
Collaborator

Build failure
Build failed or was aborted

Build succeeded
API Diff (from stable)
ℹ️ API Diff (from PR only) (please review changes)
ℹ️ Generator Diff (please review changes)

@monojenkins
Copy link
Collaborator

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

@mandel-macaque mandel-macaque merged commit add9f54 into xamarin:xcode12 Aug 20, 2020
@mandel-macaque mandel-macaque deleted the coredata-xcode12-beta4 branch August 20, 2020 00:09
[Watch (7,0), TV (14,0), Mac (11,0), iOS (14,0)]
interface NSManagedObjectsIdsChangedEventArgs {
[Export ("NSDeletedObjectIDsKey")]
NSSet DeletedObjectIds { get; }
Copy link
Member

Choose a reason for hiding this comment

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

@mandel-macaque Why doesn't this one get a Key suffix when all the other members here do?

@@ -2492,6 +2520,13 @@ interface NSConstraintConflict
NSDictionary[] ConflictingSnapshots { get; }
}

#if XAMCORE_4_0
delegate bool NSBatchInsertRequestDictionaryHandler (NSMutableDictionary<NSString, NSObject> dictionary);
Copy link
Member

Choose a reason for hiding this comment

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

Why couldn't this version be used right now?

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.

None yet

5 participants