-
-
Notifications
You must be signed in to change notification settings - Fork 47
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
Migrate from moshi to kotlinx serialization #141
Migrate from moshi to kotlinx serialization #141
Conversation
val repositoryId: Int, | ||
val issues: Array<String>? | ||
val repositoryId: Int, | ||
val issues: Array<String>? = null |
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.
Given we can give a default value, can we remove the optionality and put an empty array in case is not present?
So the user has less things to worry about, this can be an empty array or an array of issues, currently can also be null.
We would remove a state with no changes for the user
@SerialName("allow_collaboration") | ||
val allowCollaboration: Boolean?, | ||
@Json(name="allow_maintainer_to_push") | ||
@SerialName("allow_maintainer_to_push") | ||
val allowMaintainerToPush: Boolean?, |
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.
I suppose we could default those to false if not present and remove the optional
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.
Do you know what is going on with the action?
No description provided.