-
Notifications
You must be signed in to change notification settings - Fork 29.8k
Closed
flutter/plugins
#3756Labels
c: tech-debtTechnical debt, code quality, testing, etc.Technical debt, code quality, testing, etc.packageflutter/packages repository. See also p: labels.flutter/packages repository. See also p: labels.
Description
dart-flutter hhh bot identified several places where '?' is used with dynamic. This is breaking flutter_plugins shard on hhh bot:
https://logs.chromium.org/logs/dart/buildbucket/cr-buildbucket.appspot.com/8851887640643131584/+/steps/flutter_test_flutter_plugins/0/stdout
Checking project google_sign_in_web...
info - The '?' is unnecessary because 'dynamic' is nullable without it at lib/src/generated/gapi.dart:22:6 - (unnecessary_question_mark)
info - The '?' is unnecessary because 'dynamic' is nullable without it at lib/src/generated/gapi.dart:23:5 - (unnecessary_question_mark)
info - The '?' is unnecessary because 'dynamic' is nullable without it at lib/src/generated/gapi.dart:24:5 - (unnecessary_question_mark)
info - The '?' is unnecessary because 'dynamic' is nullable without it at lib/src/generated/gapi.dart:25:5 - (unnecessary_question_mark)
info - The '?' is unnecessary because 'dynamic' is nullable without it at lib/src/generated/gapi.dart:26:5 - (unnecessary_question_mark)
info - The '?' is unnecessary because 'dynamic' is nullable without it at lib/src/generated/gapiauth2.dart:71:7 - (unnecessary_question_mark)
6 issues found; analyzed 20 source files in 5.4s.
...
Checking project in_app_purchase...
info - The '?' is unnecessary because 'dynamic' is nullable without it at lib/src/in_app_purchase/in_app_purchase_connection.dart:307:9 - (unnecessary_question_mark)
1 issue found; analyzed 36 source files in 7.6s.
Metadata
Metadata
Assignees
Labels
c: tech-debtTechnical debt, code quality, testing, etc.Technical debt, code quality, testing, etc.packageflutter/packages repository. See also p: labels.flutter/packages repository. See also p: labels.