Merge pull request #27 from netglade/update-analysis

Update analysis to 8

Author: tenhobi

packages/netglade_analysis/README.md

@@ -23,7 +23,7 @@ To use the lints, add as a dev dependency in your `pubspec.yaml`:
 
 ```yaml
 dev_dependencies:
-  netglade_analysis: ^7.0.0
+  netglade_analysis: ^8.0.0
 ```
 
 Then, add an include in `analysis_options.yaml`:

packages/netglade_flutter_utils/example/pubspec.yaml

@@ -9,7 +9,7 @@ environment:
 dependencies:
   flutter:
     sdk: flutter
-  netglade_analysis: ^7.0.0
+  netglade_analysis: ^8.0.0
   netglade_flutter_utils:
     path: ..
 

packages/netglade_flutter_utils/pubspec.yaml

@@ -19,7 +19,7 @@ dev_dependencies:
   flutter_test:
     sdk: flutter
   mocktail: ^1.0.0
-  netglade_analysis: ^7.0.0
+  netglade_analysis: ^8.0.0
   test: ^1.24.6
 
 flutter:

packages/netglade_flutter_utils/test/src/hooks/route_aware_test.dart

@@ -32,15 +32,6 @@ class _UnderTestWidget extends HookWidget {
     required this.onAction,
   });
 
-  void _handlePopBtnPressed(BuildContext context) {
-    Navigator.of(context).pop();
-  }
-
-  void _handlePushNextBtnPressed(BuildContext context) {
-    // ignore: avoid-undisposed-instances, ok for test
-    unawaited(Navigator.of(context).push(MaterialPageRoute<void>(builder: (_) => const _MockPage())));
-  }
-
   @override
   Widget build(BuildContext context) {
     final state = useState<String>('??');
@@ -80,6 +71,15 @@ class _UnderTestWidget extends HookWidget {
       ],
     );
   }
+
+  void _handlePopBtnPressed(BuildContext context) {
+    Navigator.of(context).pop();
+  }
+
+  void _handlePushNextBtnPressed(BuildContext context) {
+    // ignore: avoid-undisposed-instances, ok for test
+    unawaited(Navigator.of(context).push(MaterialPageRoute<void>(builder: (_) => const _MockPage())));
+  }
 }
 
 void main() {