chore(deps): update dev dependency flutter_lints to 5.0.0, solve warnings (singerdmx#2292)

EchoEllet · web-flow · commit 5d7aee368552 · 2024-09-27T17:58:34.000+03:00
* chore(deps): update dev dependency flutter_lints to 5.0.0, solve warnings

* chore: remove redundant analysis error ignore

* chore: add avoid_web_libraries_in_flutter rule, remove empty analyzer declaration
diff --git a/analysis_options.yaml b/analysis_options.yaml
@@ -1,9 +1,5 @@
 include: package:flutter_lints/flutter.yaml
 
-analyzer:
-  errors:
-    undefined_prefixed_name: ignore
-    unsafe_html: ignore
 linter:
   rules:
     always_declare_return_types: true
@@ -33,3 +29,4 @@ linter:
     unnecessary_lambdas: true
     unnecessary_parenthesis: true
     unnecessary_string_interpolations: true
+    avoid_web_libraries_in_flutter: true
diff --git a/dart_quill_delta/pubspec.yaml b/dart_quill_delta/pubspec.yaml
@@ -15,5 +15,6 @@ dependencies:
   quiver: ^3.2.1
 
 dev_dependencies:
+  # TODO: Using 4.0.0 to not require higher version of Dart SDK
   lints: ^4.0.0
   test: ^1.24.0
diff --git a/example/pubspec.yaml b/example/pubspec.yaml
@@ -66,7 +66,7 @@ dependency_overrides:
 dev_dependencies:
   flutter_test:
     sdk: flutter
-  flutter_lints: ^4.0.0
+  flutter_lints: ^5.0.0
   build_runner: ^2.4.8
   flutter_gen_runner: ^5.4.0
 
diff --git a/flutter_quill_extensions/analysis_options.yaml b/flutter_quill_extensions/analysis_options.yaml
@@ -1,9 +1,5 @@
 include: package:flutter_lints/flutter.yaml
 
-analyzer:
-  errors:
-    undefined_prefixed_name: ignore
-    unsafe_html: ignore
 linter:
   rules:
     always_declare_return_types: true
@@ -34,3 +30,4 @@ linter:
     unnecessary_lambdas: true
     unnecessary_parenthesis: true
     unnecessary_string_interpolations: true
+    avoid_web_libraries_in_flutter: true
diff --git a/flutter_quill_extensions/lib/flutter_quill_extensions.dart b/flutter_quill_extensions/lib/flutter_quill_extensions.dart
@@ -1,4 +1,4 @@
-library flutter_quill_extensions;
+library;
 
 // ignore: implementation_imports
 import 'package:flutter_quill/src/editor_toolbar_controller_shared/clipboard/clipboard_service_provider.dart';
diff --git a/flutter_quill_extensions/pubspec.yaml b/flutter_quill_extensions/pubspec.yaml
@@ -51,7 +51,7 @@ dependencies:
 dev_dependencies:
   flutter_test:
     sdk: flutter
-  flutter_lints: ^4.0.0
+  flutter_lints: ^5.0.0
 
 flutter:
   uses-material-design: true
diff --git a/flutter_quill_test/analysis_options.yaml b/flutter_quill_test/analysis_options.yaml
@@ -1,9 +1,5 @@
 include: package:flutter_lints/flutter.yaml
 
-analyzer:
-  errors:
-    undefined_prefixed_name: ignore
-    unsafe_html: ignore
 linter:
   rules:
     always_declare_return_types: true
@@ -34,3 +30,4 @@ linter:
     unnecessary_lambdas: true
     unnecessary_parenthesis: true
     unnecessary_string_interpolations: true
+    avoid_web_libraries_in_flutter: true
diff --git a/flutter_quill_test/lib/flutter_quill_test.dart b/flutter_quill_test/lib/flutter_quill_test.dart
@@ -1,3 +1,3 @@
-library flutter_quill_test;
+library;
 
 export 'src/test/widget_tester_extension.dart';
diff --git a/flutter_quill_test/pubspec.yaml b/flutter_quill_test/pubspec.yaml
@@ -33,7 +33,7 @@ dependencies:
     sdk: flutter
 
 dev_dependencies:
-  flutter_lints: ^4.0.0
+  flutter_lints: ^5.0.0
 
 flutter:
   uses-material-design: true
diff --git a/lib/extensions.dart b/lib/extensions.dart
@@ -3,7 +3,7 @@
   'to expose certain internal APIs and should not be used directly, as it is subject to breaking changes.\n'
   'The replacement is flutter_quill_internal.dart which is also for internal use only.',
 )
-library flutter_quill.extensions;
+library;
 
 // This file contains exports that are meant to be used
 // internally and are not part of the public API as
diff --git a/lib/flutter_quill.dart b/lib/flutter_quill.dart
@@ -1,4 +1,4 @@
-library flutter_quill;
+library;
 
 export 'src/common/structs/horizontal_spacing.dart';
 export 'src/common/structs/image_url.dart';
diff --git a/lib/quill_delta.dart b/lib/quill_delta.dart
@@ -1,3 +1,3 @@
-library flutter_quill.delta;
+library;
 
 export 'package:dart_quill_delta/dart_quill_delta.dart';
diff --git a/lib/translations.dart b/lib/translations.dart
@@ -1,4 +1,4 @@
-library flutter_quill.translations;
+library;
 
 export 'src/l10n/extensions/localizations_ext.dart';
 export 'src/l10n/widgets/localizations.dart';
diff --git a/pubspec.yaml b/pubspec.yaml
@@ -65,7 +65,7 @@ dependencies:
   quill_native_bridge: '>=10.5.14 <=10.6.2'
 
 dev_dependencies:
-  flutter_lints: ^4.0.0
+  flutter_lints: ^5.0.0
   flutter_test:
     sdk: flutter
   flutter_quill_test: ^10.0.0
diff --git a/quill_native_bridge/analysis_options.yaml b/quill_native_bridge/analysis_options.yaml
@@ -1,6 +1,5 @@
 include: package:flutter_lints/flutter.yaml
 
-analyzer:
 linter:
   rules:
     always_declare_return_types: true
@@ -30,3 +29,4 @@ linter:
     unnecessary_lambdas: true
     unnecessary_parenthesis: true
     unnecessary_string_interpolations: true
+    avoid_web_libraries_in_flutter: true
diff --git a/quill_native_bridge/pubspec.yaml b/quill_native_bridge/pubspec.yaml
@@ -18,7 +18,7 @@ dependencies:
 dev_dependencies:
   flutter_test:
     sdk: flutter
-  flutter_lints: ^4.0.0
+  flutter_lints: ^5.0.0
 
 flutter:
   plugin:

Original file line number	Diff line number	Diff line change
`@@ -1,4 +1,4 @@`
`1`		`-library flutter_quill_extensions;`
	`1`	`+library;`
`2`	`2`
`3`	`3`	`// ignore: implementation_imports`
`4`	`4`	`import 'package:flutter_quill/src/editor_toolbar_controller_shared/clipboard/clipboard_service_provider.dart';`
Original file line number	Diff line number	Diff line change
`@@ -1,3 +1,3 @@`
`1`		`-library flutter_quill_test;`
	`1`	`+library;`
`2`	`2`
`3`	`3`	`export 'src/test/widget_tester_extension.dart';`
Original file line number	Diff line number	Diff line change
`@@ -3,7 +3,7 @@`
`3`	`3`	`'to expose certain internal APIs and should not be used directly, as it is subject to breaking changes.\n'`
`4`	`4`	`'The replacement is flutter_quill_internal.dart which is also for internal use only.',`
`5`	`5`	`)`
`6`		`-library flutter_quill.extensions;`
	`6`	`+library;`
`7`	`7`
`8`	`8`	`// This file contains exports that are meant to be used`
`9`	`9`	`// internally and are not part of the public API as`