-
Notifications
You must be signed in to change notification settings - Fork 13
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
PmtYaml: Parser fixes #477
Conversation
11f4ab1
to
7496797
Compare
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.
Looks good, thanks for adding this 👍 can be merged once the CI has passed.
7496797
to
2f042a4
Compare
2f042a4
to
e9b114d
Compare
Please re-review, I've changed it completely again.. @dantti Please test with your GRC parsing changes. |
* Make sure block-style maps in block-style lists are parsed correctly. This is needed to parse our existing GRC files. * Support unexpected whitespace/newlines between key and scalars as well as flow-style maps/lists. Signed-off-by: Frank Osterfeld <[email protected]>
e9b114d
to
e4597a6
Compare
Quality Gate passedIssues Measures |
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.
Still ok. If this solves @dantti issue, then I'd merge it as is.
correctly. This is needed to parse our existing GRC files.
as well as flow-style maps/lists. E.g.