photofroggy / reflex Goto Github PK

Creating event bindings could use kwargs for taking in the binding rules.

Possibly change the name from "rules" to something else? The term "arguments" may be more appropriate.

Make sure that the source parameter is not used in unit tests.

Update documentation in regards to the removal of the source parameter.

A convenience method for child classes to override on instantiation.

This isn't even slightly needed when you consider the super() method. Even so, it is admissible if it does not look like a magic method.

Such methods should be renamed to init() or inst(). init() is probably better.

GPL doesn't seem too appropriate.

ISC? Maybe.

Design change.

Remove the "source" from event bindings.

Currently, when binding an event, the programmer has to tell the event manager where the binding has come from. Reactors do this automatically, using the name attribute of the reactor.

This should not be needed as it complicates more simple use cases.

Systems using Reflex can implement this themselves if needed. Pretty trivial to be honest.