Merge pull request #7 from davidmorgan/update-lints

Update lints.

CHANGELOG.md

@@ -1,6 +1,11 @@
+## 1.2.0
+
+- Enforce `unawaited_futures`. Stop enforcing `control_flow_in_finally` and
+  `throw_in_finally`.
+
 ## 1.1.0
 
-- Moved `analysis_options.yaml` under `lib` so you can import it directly from
+- Move `analysis_options.yaml` under `lib` so you can import it directly from
   your own `analysis_options.yaml`. See `README.md` for example.
 
 ## 1.0.0

README.md

@@ -70,6 +70,10 @@ and others. See note about Flutter SDK style below.
 `avoid_as`
 does not reflect standard usage. See note about Flutter SDK style below.
 
+`control_flow_in_finally`
+does not offer enough value: people are unlikely to do this by accident,
+and there are occasional valid uses.
+
 `empty_statements`
 is superfluous, enforcing use of `dartfmt` is sufficient to make empty
  statements obvious.
@@ -80,6 +84,10 @@ is superfluous, enforcing use of `dartfmt` is sufficient to make empty
 `prefer_final_locals`
 does not reflect standard usage.
 
+`throw_in_finally`
+does not offer enough value: people are unlikely to do this by accident,
+and there are occasional valid uses.
+
 Note on Flutter SDK Style: some lints were created specifically to support
 Flutter SDK development. Flutter app developers should instead use standard
 Dart style as described in Effective Dart, and should not use these lints.

lib/analysis_options.yaml

@@ -11,13 +11,12 @@ linter:
     - avoid_relative_lib_imports
     - avoid_return_types_on_setters
     - avoid_types_as_parameter_names
-    - control_flow_in_finally
     - no_duplicate_case_values
     - prefer_contains
     - prefer_equal_for_default_values
     - prefer_is_not_empty
     - recursive_getters
-    - throw_in_finally
     - unrelated_type_equality_checks
     - use_rethrow_when_possible
+    - unawaited_futures
     - valid_regexps