-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix interpolation in events B name #17
Fix interpolation in events B name #17
Conversation
JSONTYPE = Union[None, int, str, bool, List[Any], Dict[str, Any]] | ||
from event import HostEvent | ||
|
||
# @pytest.mark.ansible_playbook('include_task/playbook_no_templating.yml') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why tests commented out? (maybe I'll review this one after we can merge the other PR and have a clean rebase of this on top)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It was uncommented in the last commit 😅 , should have squashed both commits 😄
But yeah, let me first fix #15 and once this will be merged, I will rebase this PR
6bc3596
to
20961ee
Compare
Hey 👋 , This is the little hack for interpolation issue, had a little fight with git, but it's rebased and squashed ! |
Happy to merge this, just have to resolve merge conflicts first |
Done! |
Can you rebase on top of main? I'm havingg trouble rebase-and-merging, and this repo has PR merge commits turned off. Sorry to be a pain |
5a5a617
to
c14e5d2
Compare
Lol no problem! nope wait, forgot to pull on my fork |
fix(event name):more tests fix(interpolation): Check name of B and E events revert(trace): Clean before squash fix(tests): Clean before squash
c14e5d2
to
b2cb66a
Compare
here should be good |
Fix #16
Thanks!