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

Initial SARIF support #49

Merged
merged 2 commits into from
Aug 1, 2024
Merged

Initial SARIF support #49

merged 2 commits into from
Aug 1, 2024

Conversation

frabert
Copy link
Contributor

@frabert frabert commented Jul 24, 2024

No description provided.

@frabert frabert force-pushed the sarif branch 3 times, most recently from 928b46c to 126be5f Compare July 24, 2024 10:35
@frabert
Copy link
Contributor Author

frabert commented Jul 24, 2024

Currently the generated JSON contains all of the fields even if they are empty -- trying to figure out a way to solve this issue.

@frabert frabert force-pushed the sarif branch 5 times, most recently from 858179c to e5cb1ad Compare July 25, 2024 15:48
@frabert frabert marked this pull request as ready for review July 25, 2024 15:52
@frabert frabert requested a review from xlauko as a code owner July 25, 2024 15:52
sarif/include/gap/sarif/definitions.hpp Outdated Show resolved Hide resolved
sarif/include/gap/sarif/definitions.hpp Outdated Show resolved Hide resolved
sarif/include/gap/sarif/definitions.hpp Outdated Show resolved Hide resolved
sarif/include/gap/sarif/definitions.hpp Outdated Show resolved Hide resolved
@xlauko xlauko added the enhancement New feature or request label Jul 25, 2024
@xlauko xlauko merged commit e5136e8 into main Aug 1, 2024
2 checks passed
@xlauko xlauko deleted the sarif branch August 1, 2024 12:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants