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
Fixes #16878: Fix generation of parameters and conditions from cfengine files #2822
Fixes #16878: Fix generation of parameters and conditions from cfengine files #2822
Conversation
PR updated with a new commit |
65f1a60
to
f079b4a
Compare
Test failing |
This PR is not mergeable to upper versions. |
@gpoblon there are conflits, you must rebase |
OK, squash merging this PR |
f079b4a
to
2e8e5b4
Compare
rebased, ready to merge |
@gpoblon Tests are failing |
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.
Need to fix the tests
Yes they are until the next enum PR goes in, @peckpeck is working on it. The oslib.rl does not match the new enum system yet, hence the failing tests |
OK, squash merging this PR |
2e8e5b4
to
d5e196a
Compare
OK, merging this PR |
https://issues.rudder.io/issues/16878
bug corrections : parameters partly done, and conditions fix, parenthesis conditions a bit better