-
Notifications
You must be signed in to change notification settings - Fork 9
feat: add mandatory test 6.1.27.12 #314
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
base: main
Are you sure you want to change the base?
Conversation
7f3494a
to
c912e25
Compare
Coverage after merging 313-mandatory-test-6.1.27.12 into 196-csaf-2.1
Coverage Report
|
c912e25
to
b4eeec3
Compare
Coverage after merging 313-mandatory-test-6.1.27.12 into 196-csaf-2.1
Coverage Report
|
@domachine Is this one missing a rebase? It also contains 6.1.9 and 6.1.36... |
b4eeec3
to
48cb680
Compare
Coverage after merging 313-mandatory-test-6.1.27.12 into main
Coverage Report
|
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.
LGTM
Don't forget to update the README - removing it from the known issues and placing it into the supported tests. |
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.
LGTM
…ME to the supported tests
c68402e
48cb680
to
c68402e
Compare
Coverage after merging 313-mandatory-test-6.1.27.12 into main
Coverage Report
|
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.
Let's fix that spelling mistake...
ctx.isValid = false | ||
ctx.errors.push({ | ||
instancePath: `/vulnerabilities/${index}/product_status`, | ||
message: `missing the know_effected element`, |
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.
message: `missing the know_effected element`, | |
message: `needs a know_affected element`, |
No description provided.