Skip to content
This repository was archived by the owner on Jan 14, 2025. It is now read-only.

Enforce and fix package:pedantic lints #37

Merged
merged 1 commit into from
Dec 12, 2019
Merged

Enforce and fix package:pedantic lints #37

merged 1 commit into from
Dec 12, 2019

Conversation

natebosch
Copy link
Contributor

Add an analysis_options.yaml which includes package:pedantic config.

Fix existing lints:

  • always_declare_return_types
  • annotate_overrides
  • avoid_init_to_null
  • curly_braces_in_flow_control_structures
  • empty_constructor_bodies
  • omit_local_variable_types
  • prefer_conditional_assignment
  • prefer_final_fields
  • prefer_if_null_operators
  • prefer_is_empty
  • prefer_is_not_empty
  • prefer_single_quotes
  • unnecessary_this
  • use_function_type_syntax_for_parameters

Add an `analysis_options.yaml` which includes `package:pedantic` config.

Fix existing lints:

- always_declare_return_types
- annotate_overrides
- avoid_init_to_null
- curly_braces_in_flow_control_structures
- empty_constructor_bodies
- omit_local_variable_types
- prefer_conditional_assignment
- prefer_final_fields
- prefer_if_null_operators
- prefer_is_empty
- prefer_is_not_empty
- prefer_single_quotes
- unnecessary_this
- use_function_type_syntax_for_parameters
@natebosch natebosch merged commit 213f251 into master Dec 12, 2019
@natebosch natebosch deleted the pedantic-lints branch December 12, 2019 01:15
mosuem pushed a commit to dart-lang/tools that referenced this pull request Dec 10, 2024
Add an `analysis_options.yaml` which includes `package:pedantic` config.

Fix existing lints:

- always_declare_return_types
- annotate_overrides
- avoid_init_to_null
- curly_braces_in_flow_control_structures
- empty_constructor_bodies
- omit_local_variable_types
- prefer_conditional_assignment
- prefer_final_fields
- prefer_if_null_operators
- prefer_is_empty
- prefer_is_not_empty
- prefer_single_quotes
- unnecessary_this
- use_function_type_syntax_for_parameters
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Development

Successfully merging this pull request may close these issues.

3 participants