MichaelMarner / dart-redux-remote-devtools

Remote Devtools for Dart & Flutter
https://pub.dartlang.org/packages/redux_remote_devtools
MIT License
52 stars 10 forks source link

Replace ActionEncoder, ActionDecoder, and StateEncoder with functions #11

Closed MichaelMarner closed 4 years ago

MichaelMarner commented 5 years ago

From the Dart styleguide:

AVOID defining a one-member abstract class when a simple function will do.

http://dart-lang.github.io/linter/lints/one_member_abstracts.html

ActionEncoder, ActionDecoder, and StateDecoder are all abstract classes with a single abstract method. These should be replaced with function typedefs.

This will be a breaking change, should consider moving to strict semver for versioning and releasing this as 1.0.0