analysis_options.yaml

# 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

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-lang.github.io/linter/lints/index.html.
  #
  # 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.

  # MRC: Rules below are in alphabetical order for easier maintenance
  rules:

    # MRC: Most of our uses of print are non-vital
    # Disabling for now, but we might want to re-enable in the future
    avoid_print: false

    # MRC: Flutter really wants us to use lowerCamelCase for constants
    # I won't do it for now, so disabling the linter rule
    constant_identifier_names: false

    # MRC: Not always necessary, because sometimes it's useful to have a single
    # statement right in front of the if
    curly_braces_in_flow_control_structures: false

    # MRC: This rule is about naming files as lowercase_with_underscores
    # This seems to be very useful, but the test files aren't named that way
    # I picked two files that needed renamig with this rule, so I won't disable it
    # Instead I decided to disable this rule in a per file basis for the test files
    # We might disable this in the future if it causes problems
    # This rule is currently enabled by comment, comment the line to disable
    # file_names: false

    # MRC: Too many occurrences to fix at once, disabling for now
    prefer_const_constructors: false

    # MRC: This is a bit of personal preference, but I prefer single quotes strings
    # over double quotes strings. If it causes problems or deeply annoys somebody else
    # I have no problem in disabling it
    prefer_single_quotes: true

    # MRC: Seems to cause a lot of linter warnings on localization files at the moment
    # I will have to figure out the reason for that first, then we might reenable
    unnecessary_string_escapes: false

    # MRC: Too many occurrences to fix at once, disabling it for now
    use_key_in_widget_constructors: false


# Additional information about this file can be found at
# https://dart.dev/guides/language/analysis-options