add unnecessary_underscores to recommended

Author: devoncarew

pkgs/lints/CHANGELOG.md

@@ -1,3 +1,9 @@
+## 6.0.0-wip
+
+- `recommended`:
+  - added [unnecessary_underscores] (https://github.com/dart-lang/core/issues/856)
+- Require Dart 3.7.
+
 ## 5.1.1
 
 - Updated the SDK lower bound to 3.6.

pkgs/lints/lib/recommended.yaml

@@ -63,6 +63,7 @@ linter:
     - unnecessary_string_interpolations
     - unnecessary_this
     - unnecessary_to_list_in_spreads
+    - unnecessary_underscores
     - use_function_type_syntax_for_parameters
     - use_rethrow_when_possible
     - use_super_parameters

pkgs/lints/pubspec.yaml

@@ -1,5 +1,5 @@
 name: lints
-version: 5.1.1
+version: 6.0.0-wip
 description: >
   Official Dart lint rules. Defines the 'core' and 'recommended' set of lints
   suggested by the Dart team.
@@ -11,7 +11,7 @@ topics:
   - lints
 
 environment:
-  sdk: ^3.6.0
+  sdk: ^3.7.0
 
 # NOTE: Code is not allowed in this package - do not add dependencies.
 # dependencies:

pkgs/lints/rules.md

@@ -98,6 +98,7 @@
 | [`unnecessary_string_interpolations`](https://dart.dev/lints/unnecessary_string_interpolations) | Unnecessary string interpolation. | ✅ |
 | [`unnecessary_this`](https://dart.dev/lints/unnecessary_this) | Don't access members with `this` unless avoiding shadowing. | ✅ |
 | [`unnecessary_to_list_in_spreads`](https://dart.dev/lints/unnecessary_to_list_in_spreads) | Unnecessary `toList()` in spreads. | ✅ |
+| [`unnecessary_underscores`](https://dart.dev/lints/unnecessary_underscores) | Unnecessary underscores can be removed. | ✅ |
 | [`use_function_type_syntax_for_parameters`](https://dart.dev/lints/use_function_type_syntax_for_parameters) | Use generic function type syntax for parameters. | ✅ |
 | [`use_rethrow_when_possible`](https://dart.dev/lints/use_rethrow_when_possible) | Use rethrow to rethrow a caught exception. | ✅ |
 | [`use_super_parameters`](https://dart.dev/lints/use_super_parameters) | Use super-initializer parameters where possible. | ✅ |

pkgs/lints/tool/rules.json

@@ -904,6 +904,11 @@
     "description": "Specify non-obvious type annotations for top-level and static variables.",
     "fixStatus": "hasFix"
   },
+  {
+    "name": "strict_top_level_inference",
+    "description": "Specify type annotations.",
+    "fixStatus": "hasFix"
+  },
   {
     "name": "super_goes_last",
     "description": "Place the `super` call last in a constructor initialization list.",
@@ -949,6 +954,11 @@
     "description": "Use of angle brackets in a doc comment is treated as HTML by Markdown.",
     "fixStatus": "needsFix"
   },
+  {
+    "name": "unnecessary_async",
+    "description": "No await no async.",
+    "fixStatus": "needsFix"
+  },
   {
     "name": "unnecessary_await_in_return",
     "description": "Unnecessary `await` keyword in return.",
@@ -984,6 +994,11 @@
     "description": "Avoid wrapping fields in getters and setters just to be \"safe\".",
     "fixStatus": "hasFix"
   },
+  {
+    "name": "unnecessary_ignore",
+    "description": "Don't ignore a diagnostic code that is not produced.",
+    "fixStatus": "hasFix"
+  },
   {
     "name": "unnecessary_lambdas",
     "description": "Don't create a lambda when a tear-off will do.",
@@ -1074,6 +1089,11 @@
     "description": "Unnecessary `toList()` in spreads.",
     "fixStatus": "hasFix"
   },
+  {
+    "name": "unnecessary_underscores",
+    "description": "Unnecessary underscores can be removed.",
+    "fixStatus": "hasFix"
+  },
   {
     "name": "unreachable_from_main",
     "description": "Unreachable top-level members in executable libraries.",