excaliburjs / Excalibur / 10615479262

89%

push

github

Feat: [#3157] return target on trigger (#3176)

Closes #3157

## Changes:

- The action callback of Trigger now returns the entity that triggered it.
- General cleanup of Trigger class
  - `target` is now 100% separate from `filter`, because that's what the functionality described in the documentation implied (to me).
  - Converted all the different "default value fallbacks" to a single setup.
  - Trigger used `Entity` and `Actor` interchangeably. Rewrote it to be 100% Entity (Although Entity doesn't have any collision mechanics build in, so an argument could be made to go for 100% Actor)

5707 of 7287 branches covered (78.32%)

11 of 20 new or added lines in 2 files covered. (55.0%)

12506 of 13888 relevant lines covered (90.05%)

25261.71 hits per line