ExtendRealityLtd / ExtendRealityLtd/Zinnia.Unity

Can we merge BehaviourEnabledObserver and RulesMatcher?

Open
#321 2 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
C#
Stars
321
Forks
36
PR merge metrics
No merged PRs in 30d

Description

BehaviourEnabledObserver definitely should take a Rule instead, at which point it's very similar to RulesMatcher. Perhaps the repeated invoking can be done as a standalone component that repeatedly calls an IProcessable.Process until canceled?

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.