valkyrjaio / valkyrja / 24388254153

100%

master: 100%

push

github

[Event] Update event to use listener handler and remove dispatch (#731)

# Description

Update event to use listener handler and remove dispatch.

Cross language solution.

The extra burden on the developer — writing a handler method alongside
their implementation method — is real and
acknowledged. The tradeoff is worth it: the handler is explicit,
readable, statically checkable by the compiler or
analyzer, testable in isolation, and works identically with and without
cache. The dispatch object goes away regardless
of which alternative is considered.

## Types of changes

- [X] Improvement _(non-breaking change which improves code)_
    <!-- Target the lowest major affected branch -->
- [ ] Bug fix _(non-breaking change which fixes an issue)_
    <!-- Target the lowest major affected branch -->
- [ ] New feature _(non-breaking change which adds functionality)_
    <!-- Target master -->
- [ ] Deprecation _(breaking change which removes functionality)_
    <!-- Target master -->
- [X] Breaking change _(fix or feature that would cause existing
functionality
  to change)_
<!-- Target master, unless this is a bug fix in which case let's chat
-->
- [ ] Documentation improvement
    <!-- Target appropriate branch -->

28 of 28 new or added lines in 6 files covered. (100.0%)

13120 of 13120 relevant lines covered (100.0%)

16.43 hits per line