From 5865adddf21ce1a970009dd176850eef7265ddf5 Mon Sep 17 00:00:00 2001 From: pq Date: Tue, 14 Jun 2022 12:55:53 -0700 Subject: [PATCH] stop discarding futures --- analysis_options.yaml | 1 + tool/checks/driver.dart | 4 ++-- tool/grind.dart | 8 +++++--- 3 files changed, 8 insertions(+), 5 deletions(-) diff --git a/analysis_options.yaml b/analysis_options.yaml index 003ac2caa..f51e2b020 100644 --- a/analysis_options.yaml +++ b/analysis_options.yaml @@ -39,6 +39,7 @@ linter: # - close_sinks # https://github.com/dart-lang/linter/issues/268 - comment_references - directives_ordering + - discarded_futures - invariant_booleans - join_return_with_assignment # - lines_longer_than_80_chars # under review (see #1068) diff --git a/tool/checks/driver.dart b/tool/checks/driver.dart index 9931c43cc..c636278b9 100644 --- a/tool/checks/driver.dart +++ b/tool/checks/driver.dart @@ -20,10 +20,10 @@ import 'package:path/path.dart' as path; import 'rules/visit_registered_nodes.dart'; -void main() { +Future main() async { var rules = path.normalize(io.File(path.join('lib', 'src', 'rules')).absolute.path); - Driver([VisitRegisteredNodes()]).analyze([rules]); + await Driver([VisitRegisteredNodes()]).analyze([rules]); } class Driver { diff --git a/tool/grind.dart b/tool/grind.dart index 33c32c960..a59b860f9 100644 --- a/tool/grind.dart +++ b/tool/grind.dart @@ -10,7 +10,9 @@ import 'doc.dart'; import 'rule.dart'; @Deprecated('To be removed') -void main(List args) => grind(args); +Future main(List args) async { + await grind(args); +} Iterable get sourcePaths => sources.map((dir) => dir.path); @@ -26,10 +28,10 @@ Iterable get sources => existingSourceDirs.expand((dir) { }); @Task('Generate lint rule docs.') -void docs() { +Future docs() async { var args = context.invocation.arguments; var dir = args.getOption('dir'); - generateDocs(dir); + await generateDocs(dir); } @Task('Format linter sources.')