Add analyze and format GitHub action (#9)

.github/workflows/analyze-format.yml

@@ -0,0 +1,33 @@
+name: Analyze and format
+
+on:
+    # Trigger the workflow on pushes to the main and feature branches as well as PRs targeting them.
+    push:
+        branches: [main, feature/**]
+    pull_request:
+        branches: [main, feature/**]
+
+    # Allows us to run the workflow manually from the Actions tab
+    workflow_dispatch:
+
+env:
+    flutter_version: 3.13.6
+
+jobs:
+    analyze-format:
+        runs-on: ubuntu-22.04
+        steps:
+            - name: Checkout repository
+              uses: actions/checkout@v4
+
+            - name: Setup Flutter
+              uses: subosito/flutter-action@v2
+              with:
+                flutter-version: ${{ env.flutter_version }}
+                channel: 'stable'
+                cache: true
+            - name: Analyze (lint)
+              run: flutter analyze
+
+            - name: Check format
+              run: dart format --output=none --set-exit-if-changed . -l 150

.gitignore

@@ -16,10 +16,8 @@ migrate_working_dir/
 *.iws
 .idea/
 
-# The .vscode folder contains launch configuration and tasks you configure in
-# VS Code which you may wish to be included in version control, so this line
-# is commented out by default.
-#.vscode/
+# VS Code related
+.vscode/
 
 # Flutter/Dart/Pub related
 **/doc/api/

lib/screens/home/screen.dart

@@ -9,15 +9,15 @@ class HomeScreen extends StatelessWidget {
   @override
   Widget build(BuildContext context) {
     final tacLastCheckedAt = context.select((AppState state) => state.termsAndConditionsLastVerifiedAt);
-    print('tacLastCheckedAt: $tacLastCheckedAt');
+    //print('tacLastCheckedAt: $tacLastCheckedAt');
 
     // Temporary...
     final state = context.watch<AppState>();
     var lastAcceptedVersion = state.sharedPreferences.termsAndConditionsAcceptedVersion;
     // print('lastAcceptedVersion: $lastAcceptedVersion');
 
     // Force recheck after 1 min.
-    print('diff: ${DateTime.now().difference(tacLastCheckedAt).inMinutes}');
+    // print('diff: ${DateTime.now().difference(tacLastCheckedAt).inMinutes}');
     if (DateTime.now().difference(tacLastCheckedAt).inMinutes > 1) {
       return const RefreshTermsAndConditionsScreen();
     }

lib/screens/terms_and_conditions/refresh_screen.dart

@@ -33,7 +33,7 @@ class RefreshTermsAndConditionsScreen extends StatelessWidget {
             if (currentTac != null) {
               // TODO It feels wrong to have this business logic in the widget builder.
               if (currentTac.version == acceptedTacVersion) {
-                print('already accepted; dismissing');
+                // print('already accepted; dismissing');
                 // Current T&C is already accepted; update the T&C check time to make the home screen replace this widget.
                 Future.microtask(() => _markCheckPerformed(context));
               } else {

lib/screens/terms_and_conditions/widget.dart

@@ -4,12 +4,9 @@ class ToggleAcceptedWidget extends StatelessWidget {
   final bool isAccepted;
   final Function(bool) setAccepted;
 
-  ToggleAcceptedWidget(
-      {super.key, required this.isAccepted, required this.setAccepted});
+  ToggleAcceptedWidget({super.key, required this.isAccepted, required this.setAccepted});
 
-  final MaterialStateProperty<Icon?> icon =
-      MaterialStateProperty.resolveWith<Icon?>(
-          (states) => Icon(_iconData(states)));
+  final MaterialStateProperty<Icon?> icon = MaterialStateProperty.resolveWith<Icon?>((states) => Icon(_iconData(states)));
 
   static IconData _iconData(Set<MaterialState> states) {
     if (states.contains(MaterialState.selected)) {

lib/state.dart

@@ -15,7 +15,8 @@ final testnet = Network(
   ),
 );
 
-class AppSharedPreferences { // TODO: Extend ChangeNotifier?
+// TODO: Extend ChangeNotifier?
+class AppSharedPreferences {
   static const _tacAcceptedVersionKey = 'tac:accepted_version';
 
   final SharedPreferences _prefs;