-
-
Notifications
You must be signed in to change notification settings - Fork 7
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
feat: Add batch and increment double operation #88
Conversation
Still need to fix mix array batch operations |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tailor (reported by Codacy) found more than 10 potential problems in the proposed changes. Check the Files changed tab for more details.
Codecov Report
@@ Coverage Diff @@
## main #88 +/- ##
==========================================
+ Coverage 90.50% 90.97% +0.47%
==========================================
Files 169 172 +3
Lines 15123 15209 +86
==========================================
+ Hits 13687 13837 +150
+ Misses 1436 1372 -64
... and 20 files with indirect coverage changes Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
An operation that removes a set of objects from a field. | ||
*/ | ||
public struct ParseOperationRemove<T>: ParseOperationable where T: Codable { | ||
public var __op: ParseOperationCommand = .remove // swiftlint:disable:this identifier_name |
Check warning
Code scanning / Tailor (reported by Codacy)
Variable names should be lowerCamelCase
An operation that removes a set of relations on an array field. | ||
*/ | ||
public struct ParseOperationRemoveRelation<T>: ParseRelationOperationable where T: ParseObject { | ||
public var __op: ParseOperationCommand = .removeRelation // swiftlint:disable:this identifier_name |
Check warning
Code scanning / Tailor (reported by Codacy)
Variable names should be lowerCamelCase
} | ||
|
||
enum CodingKeys: String, CodingKey { | ||
case __op |
Check warning
Code scanning / Tailor (reported by Codacy)
Enum case names should be lowerCamelCase
- note: To append `ParseOperationable` operations of a different type, use `appendOperations()` | ||
after creating an instance of `ParseOperationBatch`. | ||
*/ | ||
public init<T>(operations: [T]) where T: ParseOperationable { |
Check warning
Code scanning / Tailor (reported by Codacy)
Opening brace should have exactly one space before it
@@ -10,7 +10,7 @@ | |||
|
|||
/// A type used to create internal fields for `ParseSchema`. | |||
public struct ParseField: ParseTypeable { | |||
var __op: Operation? // swiftlint:disable:this identifier_name | |||
var __op: ParseOperationCommand? // swiftlint:disable:this identifier_name |
Check warning
Code scanning / Tailor (reported by Codacy)
Variable names should be lowerCamelCase
- note: A field can be incremented/decremented by a positive/negative value. | ||
*/ | ||
public struct ParseOperationIncrement: ParseOperationable { | ||
public var __op: ParseOperationCommand = .increment // swiftlint:disable:this identifier_name |
Check warning
Code scanning / Tailor (reported by Codacy)
Variable names should be lowerCamelCase
- note: A field can be incremented/decremented by a positive/negative value. | ||
*/ | ||
public struct ParseOperationIncrementDouble: ParseOperationable { | ||
public var __op: ParseOperationCommand = .increment // swiftlint:disable:this identifier_name |
Check warning
Code scanning / Tailor (reported by Codacy)
Variable names should be lowerCamelCase
New Pull Request Checklist
Issue Description
Approach
TODOs before merging