Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: update all_lint_rules.yaml with Flutter 3.10 / Dart 3.0 #2

Merged
merged 2 commits into from May 12, 2023

Conversation

riscait
Copy link
Member

@riscait riscait commented May 12, 2023

Flutter 3.10 / Dart 3で追加されたリントルール

  • deprecated_member_use_from_same_package
    同パッケージ内で非推奨メンバーが使用された場合に警告
  • implicit_reopen
    意図しないクラスの拡張等を防ぐために@ReOpenを使用することを強制させる
  • invalid_case_patterns
    Dart 3では不正となる caseパターンを警告
  • matching_super_parameters
    super パラメータが対応する super constructor のパラメータ名と一致しない場合に警告
  • no_literal_bool_comparisons
    非NULLなbool型に対する等号演算にtrueやfalseを使用すると警告
  • type_literal_in_constant_pattern
    パターンにおいて、 == ではなく型リテラルを使用すると警告
  • unnecessary_breaks
    switch-case において、空ケース以外の不要な break を警告

削除されたリントルール

  • enable_null_safety

add new rules to `all_lint_rules.yaml`
- `deprecated_member_use_from_same_package`
- `implicit_reopen`
- `invalid_case_patterns`
- `matching_super_parameters`
- `no_literal_bool_comparisons`
- `type_literal_in_constant_pattern`
- `unnecessary_breaks`

remove from `all_lint_rules.yaml`
- `enable_null_safety`
@riscait riscait requested a review from naipaka May 12, 2023 01:23
Copy link

@naipaka naipaka left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@riscait riscait changed the title Flutter 3.10 dart 3.0 feat: update all_lint_rules.yaml with Flutter 3.10 / Dart 3.0 May 12, 2023
@riscait riscait merged commit 1f4c5ab into main May 12, 2023
@riscait riscait deleted the flutter-3.10-dart-3.0 branch May 12, 2023 01:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants