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

refactor: update linting rules #163

Merged
merged 1 commit into from
Aug 20, 2024
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 6 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -7,6 +7,12 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0

## [Unreleased]

### Changed

- update linting rules —
[162](https://github.com/dartoos-dev/json_cache/issues/154).


## [3.0.2] - 2024-08-19

### Changed
Expand Down
16 changes: 8 additions & 8 deletions analysis_options.yaml
Original file line number Diff line number Diff line change
@@ -1,17 +1,17 @@
include: package:lint/package.yaml

analyzer:
strong-mode:
implicit-casts: false

exclude:
- '**.freezed.dart'
- '**.g.dart'
linter:
rules:
# Make constructors the first thing in every class
sort_unnamed_constructors_first: true
sort_constructors_first: true
avoid_catches_without_on_clauses: true
avoid_equals_and_hash_code_on_mutable_classes: true
cancel_subscriptions: true
# Good packages document everything
public_member_api_docs: true
always_declare_return_types: true
cancel_subscriptions: true
close_sinks: true
only_throw_errors: true
package_api_docs: true
package_api_docs: true
2 changes: 1 addition & 1 deletion pubspec.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -22,6 +22,6 @@ dev_dependencies:
flutter_test:
sdk: flutter
hive_test: ^1.0.1
lint: ^2.0.1
lint: ^2.3.0
mocktail: ^1.0.0
test: ^1.24.9