feat(ensure): add ensure_states feature #296
Conversation
| # Switch1.on | ||
| # Switch2.on | ||
| # end | ||
| def ensure_states |
There was a problem hiding this comment.
We have this same code in rules as well. In logging I was going to pull this out into a common element.
Thinking along the lines of
thread_local(key: value) do
end
Don't need to add it to the PR, but WDYT?
There was a problem hiding this comment.
yeah, seems like a decent de-duplication
ccutrer
force-pushed
the
ensure_states
branch
from
0cbd5d3 to
b06385b
Compare
|
LGTM. |
|
🤦 I thought I had docs for this one. Probably cause I spent a good amount of time writing very good tests. Yeah, we can merge and add docs later. |
|
Ok, I can't write docs tonight.. But I will prep a PR with docs early tomorrow then merge the two of them right after each other. |
|
🎉 This PR is included in version 4.4.0 🎉 The release is available on:
Your semantic-release bot 📦🚀 |
feat(ensure): add ensure_states feature Former-commit-id: 71ac48b
closes #275