-
Notifications
You must be signed in to change notification settings - Fork 309
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 to null-safety #263
Migrate to null-safety #263
Conversation
final customWeekdayBuilder = this.customWeekdayBuilder; | ||
return customWeekdayBuilder != 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.
Why don't you just compare with this.customWeekdayBuilder;
?
return this.customWeekdayBuilder != 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.
@WilliamCunhaCardoso Looks like @IchordeDionysos didn't want to use this.customWeekdayBuilder
twice in the below lines.
width: markedDateMoreShowTotal ? 18 : null, | ||
height: markedDateMoreShowTotal ? 18 : null, | ||
decoration: markedDateMoreCustomDecoration == null | ||
? new BoxDecoration( |
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.
new
cast is not necessary
? new BoxDecoration( | |
? BoxDecoration( |
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.
Since this is not a critical one, will be fine here.
Very nice work. Why does it still on draft? |
Does this PR also solves this? #250 |
It's been 10 days since the PR. Any news about this or sound null-safety? |
@IchordeDionysos Is this ready for review? |
@hyochan yes sorry :) |
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.
Very nicely done 💯 Thanks for your contribution 🙌
Migrates the package to support null-safety. 🚀
Closes #260.