-
Notifications
You must be signed in to change notification settings - Fork 555
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
feat: replace YAML parser with library call [CC-1012] #2260
Conversation
9b1b564
to
b473e27
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.
Nice! Reviewer note: I haven't read the cloud config parser code again, I'm relying on our tests in both of these repos catching any regressions.
That is a really massive diff to the package-lock though, are we sure that's done the right thing?
00b9fe2
to
e8ff8ae
Compare
Good shout on the package-lock.json. I followed the instructions in this thread to fix it: https://snyk.slack.com/archives/C1NREQSG0/p1631814339032800 |
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.
maybe remove the ^
symbol from "@snyk/cloud-config-parser": "^1.11.0"
to avoid breaking changes unless it's a desired behavior from your team
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.
nice one @teodora-sandu and amazing PR description 🔥
e8ff8ae
to
0d75edc
Compare
What does this PR do?
This PR replaces the
yaml
library with a call to@snyk/cloud-config-parser
, where we have moved our customised YAML parsing logic. This way when we have parsing issues that we need to fix, we can fix them in just one place.How should this be manually tested?
Run:
npm run build
snyk-dev iac test ./test/fixtures/iac/kubernetes/pod-privileged.yaml
snyk-dev iac test ./test/fixtures/iac/kubernetes/pod-valid.json
What are the relevant tickets?
https://snyksec.atlassian.net/browse/CC-1012
Screenshots