Remove nullable getter for ResponsiveValue#123
Remove nullable getter for ResponsiveValue#123StrDragos wants to merge 1 commit intoCodelessly:masterfrom
Conversation
|
Thank you for the PR |
|
Really interesting API decisions with ResponsiveValue. The choice was between making The latter was chosen for 2 reasons.
The drawback with this approach is an explicit nonnull cast |
The problem with this approach is that if you pass the |
Remove the null possibility of a
ReponsiveValueon thevaluegetter. It is not needed sinceResponsiveValuealso receives a default value.Will also address:
ResponsiveValue.value should non-nullable #119