251: Configure linting, formatting and typechecking

.github/hooks/pre-commit

@@ -0,0 +1,21 @@
+#!/bin/sh
+
+#
+# Install this using:
+# ln -s ../../.github/hooks/pre-commit .git/hooks/pre-commit
+#
+
+DART_FILES=$(git diff --cached --name-only --diff-filter=ACMR "*.dart" | sed 's| |\\ |g')
+
+if [ ! -z "$DART_FILES" ]; then
+  if [ ! -x "$(command -v fvm)" ]; then
+    echo "You need to install fvm in order to commit and format files"
+    exit 1
+  fi
+
+  # Prettify all selected files
+  echo "$DART_FILES" | xargs fvm dart format -l 120
+
+  # Add back the modified/prettified files to staging
+  echo "$DART_FILES" | xargs git add
+fi

analysis_options.yaml

@@ -1,28 +1,20 @@
-# This file configures the analyzer, which statically analyzes Dart code to
-# check for errors, warnings, and lints.
-#
-# The issues identified by the analyzer are surfaced in the UI of Dart-enabled
-# IDEs (https://dart.dev/tools#ides-and-editors). The analyzer can also be
-# invoked from the command line by running `flutter analyze`.
-
-# The following line activates a set of recommended lints for Flutter apps,
-# packages, and plugins designed to encourage good coding practices.
 include: package:flutter_lints/flutter.yaml
 
+analyzer:
+  exclude: [build/**]
+  errors:
+    always_use_package_imports: error
+    directives_ordering: error
+    prefer_single_quotes: error
+    require_trailing_commas: error
+  language:
+    strict-casts: true
+    strict-raw-types: true
+    strict-inference: true
+
 linter:
-  # The lint rules applied to this project can be customized in the
-  # section below to disable rules from the `package:flutter_lints/flutter.yaml`
-  # included above or to enable additional rules. A list of all available lints
-  # and their documentation is published at https://dart.dev/lints.
-  #
-  # Instead of disabling a lint rule for the entire project in the
-  # section below, it can also be suppressed for a single line of code
-  # or a specific dart file by using the `// ignore: name_of_lint` and
-  # `// ignore_for_file: name_of_lint` syntax on the line or in the file
-  # producing the lint.
   rules:
-    # avoid_print: false  # Uncomment to disable the `avoid_print` rule
-    # prefer_single_quotes: true  # Uncomment to enable the `prefer_single_quotes` rule
-
-# Additional information about this file can be found at
-# https://dart.dev/guides/language/analysis-options
+    - always_use_package_imports
+    - directives_ordering
+    - prefer_single_quotes
+    - require_trailing_commas

lib/main.dart

@@ -5,7 +5,7 @@ void main() {
 }
 
 class MyApp extends StatelessWidget {
-  const MyApp({Key? key}) : super(key: key);
+  const MyApp({super.key});
 
   @override
   Widget build(BuildContext context) {