Skip to content
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

writer-json-sarif: add tool to rule properties #147

Merged
merged 4 commits into from
Oct 19, 2023

Conversation

kdudka
Copy link
Member

@kdudka kdudka commented Oct 17, 2023

... to ease SARIF import into Defect Dojo

Resolves: https://issues.redhat.com/browse/OSH-355

No changes in behavior intended with this commit.
No changes in behavior intended with this commit.
No changes in behavior intended with this commit, except for reordering
of unordered items in a JSON object.
@kdudka kdudka self-assigned this Oct 17, 2023
@kdudka
Copy link
Member Author

kdudka commented Oct 18, 2023

@hanchuntao We have already a confirmation that the change improved the import into Defect Dojo. We would just need another pair of eyes to have a look at the code changes. Could you please take the review of this?

@hanchuntao
Copy link

hanchuntao commented Oct 19, 2023

LGTM, please rebase the code

@kdudka
Copy link
Member Author

kdudka commented Oct 19, 2023

@hanchuntao Thanks for review!

@kdudka kdudka closed this in bb1634e Oct 19, 2023
@kdudka kdudka merged commit bb1634e into csutils:main Oct 19, 2023
32 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants