🚨 Add require_trailing_commas lint

Author: AlexV525

analysis_options.yaml

@@ -169,6 +169,7 @@ linter:
     # - provide_deprecation_message # not yet tested
     # - public_member_api_docs # enabled on a case-by-case basis; see e.g. packages/analysis_options.yaml
     - recursive_getters
+    - require_trailing_commas
     # - sized_box_for_whitespace # not yet tested
     - slash_for_doc_comments
     # - sort_child_properties_last # not yet tested

example/lib/main.dart

@@ -18,9 +18,9 @@ PackageInfo? packageInfo;
 
 void main() {
   runApp(MyApp());
-  SystemChrome.setSystemUIOverlayStyle(SystemUiOverlayStyle.dark.copyWith(
-    statusBarColor: Colors.transparent,
-  ));
+  SystemChrome.setSystemUIOverlayStyle(
+    SystemUiOverlayStyle.dark.copyWith(statusBarColor: Colors.transparent),
+  );
   AssetPicker.registerObserve();
 }
 
@@ -62,6 +62,9 @@ class NoGlowScrollBehavior extends ScrollBehavior {
 
   @override
   Widget buildViewportChrome(
-          BuildContext context, Widget child, AxisDirection axisDirection) =>
+    BuildContext context,
+    Widget child,
+    AxisDirection axisDirection,
+  ) =>
       child;
 }

lib/src/delegates/asset_picker_builder_delegate.dart

@@ -472,11 +472,13 @@ abstract class AssetPickerBuilderDelegate<Asset, Path> {
         bottom: context.bottomPadding,
       ),
       color: theme.primaryColor.withOpacity(isAppleOS ? 0.90 : 1),
-      child: Row(children: <Widget>[
-        if (!isSingleAssetMode || !isAppleOS) previewButton(context),
-        if (isAppleOS) const Spacer(),
-        if (isAppleOS) confirmButton(context),
-      ]),
+      child: Row(
+        children: <Widget>[
+          if (!isSingleAssetMode || !isAppleOS) previewButton(context),
+          if (isAppleOS) const Spacer(),
+          if (isAppleOS) confirmButton(context),
+        ],
+      ),
     );
     if (isPermissionLimited) {
       child = Column(

lib/src/provider/asset_picker_provider.dart

@@ -367,22 +367,24 @@ class DefaultAssetPickerProvider
 
   @override
   Future<void> switchPath([AssetPathEntity? pathEntity]) async {
-    assert(() {
-      if (_currentPathEntity == null && pathEntity == null) {
-        throw FlutterError.fromParts(<DiagnosticsNode>[
-          ErrorSummary('Empty $AssetPathEntity was switched.'),
-          ErrorDescription(
-            'Neither currentPathEntity nor pathEntity is non-null, which makes '
-            'this method useless.',
-          ),
-          ErrorHint(
-            'You need to pass a non-null $AssetPathEntity or call this method '
-            'when currentPathEntity is not null.',
-          ),
-        ]);
-      }
-      return true;
-    }());
+    assert(
+      () {
+        if (_currentPathEntity == null && pathEntity == null) {
+          throw FlutterError.fromParts(<DiagnosticsNode>[
+            ErrorSummary('Empty $AssetPathEntity was switched.'),
+            ErrorDescription(
+              'Neither currentPathEntity nor pathEntity is non-null, '
+              'which makes this method useless.',
+            ),
+            ErrorHint(
+              'You need to pass a non-null $AssetPathEntity '
+              'or call this method when currentPathEntity is not null.',
+            ),
+          ]);
+        }
+        return true;
+      }(),
+    );
     if (_currentPathEntity == null && pathEntity == null) {
       return;
     }

lib/src/widget/custom_checkbox.dart

@@ -385,17 +385,23 @@ class _CustomCheckboxState extends State<CustomCheckbox>
         break;
       case MaterialTapTargetSize.shrinkWrap:
         size = const Size(
-            kMinInteractiveDimension - 8.0, kMinInteractiveDimension - 8.0);
+          kMinInteractiveDimension - 8.0,
+          kMinInteractiveDimension - 8.0,
+        );
         break;
     }
     size += effectiveVisualDensity.baseSizeAdjustment;
     final BoxConstraints additionalConstraints = BoxConstraints.tight(size);
     final MouseCursor effectiveMouseCursor =
         MaterialStateProperty.resolveAs<MouseCursor?>(
-                widget.mouseCursor, _states) ??
+              widget.mouseCursor,
+              _states,
+            ) ??
             themeData.checkboxTheme.mouseCursor?.resolve(_states) ??
             MaterialStateProperty.resolveAs<MouseCursor>(
-                MaterialStateMouseCursor.clickable, _states);
+              MaterialStateMouseCursor.clickable,
+              _states,
+            );
     // Colors need to be resolved in selected and non selected states separately
     // so that they can be lerped between.
     final Set<MaterialState> activeStates = _states
@@ -1285,12 +1291,23 @@ abstract class RenderToggleable extends RenderConstrainedBox {
   @override
   void debugFillProperties(DiagnosticPropertiesBuilder properties) {
     super.debugFillProperties(properties);
-    properties.add(FlagProperty('value',
-        value: value, ifTrue: 'checked', ifFalse: 'unchecked', showName: true));
-    properties.add(FlagProperty('isInteractive',
+    properties.add(
+      FlagProperty(
+        'value',
+        value: value,
+        ifTrue: 'checked',
+        ifFalse: 'unchecked',
+        showName: true,
+      ),
+    );
+    properties.add(
+      FlagProperty(
+        'isInteractive',
         value: isInteractive,
         ifTrue: 'enabled',
         ifFalse: 'disabled',
-        defaultValue: true));
+        defaultValue: true,
+      ),
+    );
   }
 }