solid_lints

solid_lints copied to clipboard

sufftea

New rule: consider_making_a_member_private (if its not used outside)

2

This'll help with dead_code BAD: ```dart class X { // only used internally void x() { ... } } ``` GOOD: ```dart class X { // only used internally void...

danylo-safonov-solid

Update avoid_debug_print suggested fix

1

See here more: https://github.com/flutter/flutter/issues/147141 We might want just to provide our own utility to do logging? Or provide better example that just makes sure that debugPrint is guarded by kDebugMode?

illia-romanenko

We need to refactor the current package structure to make it consistent over different rules. The package structure should be the following for all the rules: ``` - lints -...

solid-vovabeloded

Ignore methods/classes

1

In few rules we want to ignore certain methods/classes, and we want to extract and reuse this code. Common issues is support of generics, etc.

illia-romanenko

avoid_non_null_assertion support IMap from FIC package as valid

2

Hi, currently using https://pub.dev/packages/fast_immutable_collections and with these lint give a warning for every IMap call. Is feasible to add the exception? or maybe add the possibility to add valid types...

hectorAguero

…s` lint

Viktor-Mudrytskyi

 

danylo-safonov-solid

New rule: feature_envy

1

https://refactoring.guru/smells/feature-envy https://github.com/troessner/reek/blob/master/lib/reek/smell_detectors/feature_envy.rb https://github.com/tsantalis/JDeodorant/blob/master/src/gr/uom/java/jdeodorant/refactoring/views/FeatureEnvy.java

danylo-safonov-solid

I'm currently using: ```yaml - avoid_late_keyword: allow_initialized: true ignored_types: - AnimationController - StreamSubscription - Timer ``` Timer is working fine and AnimationController too, but StreamSubscription is not working and here...

arthurbcd