-
Notifications
You must be signed in to change notification settings - Fork 24
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
fix: update key and lints #1155
Conversation
@@ -13,14 +13,13 @@ class DecoratedEventWidget extends StatelessWidget { | |||
final EdgeInsets padding; | |||
|
|||
const DecoratedEventWidget._( | |||
{Key? key, | |||
{super.key, |
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.
Is this what dart fmt
spits out? It's fine if so, but a little surprised.
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.
Yeah dart format nor the analyzer throw me any errors at all, if I do try and correct it to the previous way the latest analyzer warns me to change it again.
@@ -36,11 +36,9 @@ class TwitchPredictionEventWidget extends StatelessWidget { | |||
|
|||
class _TwitchOutcomeWidget extends StatelessWidget { | |||
const _TwitchOutcomeWidget( | |||
{Key? key, | |||
required this.outcome, | |||
{required this.outcome, |
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.
Missing super.key
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.
A value for optional parameter 'key' isn't ever given.
Try removing the unused parameter.
@@ -11,8 +11,7 @@ class _DurationWidget extends StatelessWidget { | |||
final DateTime from; | |||
final TextStyle? style; | |||
|
|||
const _DurationWidget({Key? key, this.style, required this.from}) | |||
: super(key: key); | |||
const _DurationWidget({this.style, required this.from}); |
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.
Missing super.key
.
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.
A value for optional parameter 'key' isn't ever given.
Try removing the unused parameter.
No description provided.