chore: update lint rules

This commit updates the lint rules in `all_lint_rules.yaml` and adjusts the `analysis_options.yaml` files accordingly.

The following changes were made:

- **Enabled new lint rules:** A significant number of new lint rules from the Flutter and Dart linters have been added to `all_lint_rules.yaml`.
- **Removed deprecated/conflicting lint rules:**
    - `always_require_non_null_named_parameters`
    - `avoid_returning_null`
    - `avoid_returning_null_for_future`
    - `iterable_contains_unrelated_type` (replaced by stricter checks)
    - `list_remove_unrelated_type` (replaced by stricter checks)
    - `package_api_docs`
    - `prefer_equal_for_default_values`
- **Adjusted `analysis_options.yaml`:**
    - Added `packages/*/lib/**/*.freezed.dart` to the exclude list.
    - Disabled `document_ignores` as it's considered redundant.
    - Disabled `specify_nonobvious_local_variable_types` and `specify_nonobvious_property_types` due to conflict with `omit_local_variable_types`.
    - Temporarily disabled `public_member_api_docs` (moved from `avoid_print`).
- **Adjusted `sample_app/analysis_options.yaml`:**
    - Added `lib/**/*.g.dart` and `lib/**/*.freezed.dart` to the exclude list (as a temporary workaround for an issue with root analysis file exclusion).

Author: xsahil03x

all_lint_rules.yaml

@@ -3,10 +3,10 @@ linter:
     - always_declare_return_types
     - always_put_control_body_on_new_line
     - always_put_required_named_parameters_first
-    - always_require_non_null_named_parameters
     - always_specify_types
     - always_use_package_imports
     - annotate_overrides
+    - annotate_redeclares
     - avoid_annotating_with_dynamic
     - avoid_bool_literals_in_conditional_expressions
     - avoid_catches_without_on_clauses
@@ -20,6 +20,7 @@ linter:
     - avoid_field_initializers_in_const_classes
     - avoid_final_parameters
     - avoid_function_literals_in_foreach_calls
+    - avoid_futureor_void
     - avoid_implementing_value_types
     - avoid_init_to_null
     - avoid_js_rounded_ints
@@ -32,8 +33,6 @@ linter:
     - avoid_relative_lib_imports
     - avoid_renaming_method_parameters
     - avoid_return_types_on_setters
-    - avoid_returning_null
-    - avoid_returning_null_for_future
     - avoid_returning_null_for_void
     - avoid_returning_this
     - avoid_setters_without_getters
@@ -61,12 +60,15 @@ linter:
     - constant_identifier_names
     - control_flow_in_finally
     - curly_braces_in_flow_control_structures
+    - dangling_library_doc_comments
     - depend_on_referenced_packages
     - deprecated_consistency
+    - deprecated_member_use_from_same_package
     - diagnostic_describe_all_properties
     - directives_ordering
     - discarded_futures
     - do_not_use_environment
+    - document_ignores
     - empty_catches
     - empty_constructor_bodies
     - empty_statements
@@ -76,32 +78,41 @@ linter:
     - flutter_style_todos
     - hash_and_equals
     - implementation_imports
-    - iterable_contains_unrelated_type
+    - implicit_call_tearoffs
+    - implicit_reopen
+    - invalid_case_patterns
+    - invalid_runtime_check_with_js_interop_types
     - join_return_with_assignment
     - leading_newlines_in_multiline_strings
+    - library_annotations
     - library_names
     - library_prefixes
     - library_private_types_in_public_api
     - lines_longer_than_80_chars
-    - list_remove_unrelated_type
     - literal_only_boolean_expressions
+    - matching_super_parameters
+    - missing_code_block_language_in_doc_comment
     - missing_whitespace_between_adjacent_strings
     - no_adjacent_strings_in_list
     - no_default_cases
     - no_duplicate_case_values
     - no_leading_underscores_for_library_prefixes
     - no_leading_underscores_for_local_identifiers
+    - no_literal_bool_comparisons
     - no_logic_in_create_state
     - no_runtimeType_toString
+    - no_self_assignments
+    - no_wildcard_variable_uses
     - non_constant_identifier_names
     - noop_primitive_operations
     - null_check_on_nullable_type_parameter
     - null_closures
     - omit_local_variable_types
+    - omit_obvious_local_variable_types
+    - omit_obvious_property_types
     - one_member_abstracts
     - only_throw_errors
     - overridden_fields
-    - package_api_docs
     - package_names
     - package_prefixed_library_names
     - parameter_assignments
@@ -117,7 +128,6 @@ linter:
     - prefer_constructors_over_static_methods
     - prefer_contains
     - prefer_double_quotes
-    - prefer_equal_for_default_values
     - prefer_expression_function_bodies
     - prefer_final_fields
     - prefer_final_in_for_each
@@ -157,20 +167,31 @@ linter:
     - sort_constructors_first
     - sort_pub_dependencies
     - sort_unnamed_constructors_first
+    - specify_nonobvious_local_variable_types
+    - specify_nonobvious_property_types
+    - strict_top_level_inference
+    - switch_on_type
     - test_types_in_equals
     - throw_in_finally
     - tighten_type_of_initializing_formals
     - type_annotate_public_apis
     - type_init_formals
+    - type_literal_in_constant_pattern
     - unawaited_futures
+    - unintended_html_in_doc_comment
+    - unnecessary_async
     - unnecessary_await_in_return
     - unnecessary_brace_in_string_interps
+    - unnecessary_breaks
     - unnecessary_const
     - unnecessary_constructor_name
     - unnecessary_final
     - unnecessary_getters_setters
+    - unnecessary_ignore
     - unnecessary_lambdas
     - unnecessary_late
+    - unnecessary_library_directive
+    - unnecessary_library_name
     - unnecessary_new
     - unnecessary_null_aware_assignments
     - unnecessary_null_aware_operator_on_extension_on_nullable
@@ -185,9 +206,11 @@ linter:
     - unnecessary_string_interpolations
     - unnecessary_this
     - unnecessary_to_list_in_spreads
+    - unnecessary_unawaited
+    - unnecessary_underscores
     - unreachable_from_main
     - unrelated_type_equality_checks
-    - unsafe_html
+    - unsafe_variance
     - use_build_context_synchronously
     - use_colored_box
     - use_decorated_box
@@ -199,6 +222,7 @@ linter:
     - use_key_in_widget_constructors
     - use_late_for_private_fields_and_variables
     - use_named_constants
+    - use_null_aware_elements
     - use_raw_strings
     - use_rethrow_when_possible
     - use_setters_to_change_properties
@@ -207,5 +231,6 @@ linter:
     - use_super_parameters
     - use_test_throws_matchers
     - use_to_and_as_if_applicable
+    - use_truncating_division
     - valid_regexps
-    - void_checks
+    - void_checks

analysis_options.yaml

@@ -10,11 +10,10 @@ analyzer:
     # We explicitly enabled even conflicting rules and are fixing the conflict
     # in this file.
     included_file_warning: ignore
-
-    todo: ignore
   exclude:
     # exclude all the generated files
     - packages/*/lib/**/*.g.dart
+    - packages/*/lib/**/*.freezed.dart
 
 linter:
   rules:
@@ -93,6 +92,12 @@ linter:
     # There are situations where we use default in enums on purpose
     no_default_cases: false
 
-    # Temporarily disabled to find more important issues
+    # Redundant and not required
+    document_ignores: false
+
+    # Conflicts with `omit_local_variable_types`
+    specify_nonobvious_local_variable_types: false
+    specify_nonobvious_property_types: false
+
+    # TODO: Temporarily disabled to find more important issues
     public_member_api_docs: false
-    avoid_print: false

sample_app/analysis_options.yaml

@@ -1,5 +1,12 @@
 include: ../analysis_options.yaml
 
+analyzer:
+  # TODO: not working if added on the root analysis file
+  exclude:
+    # exclude all the generated files
+    - lib/**/*.g.dart
+    - lib/**/*.freezed.dart
+
 linter:
   rules:
     cascade_invocations: false