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.

Sign up for GitHub

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

Jump to bottom

Add infrastructure to support SARIF output #701

Merged
merged 7 commits into from
Sep 3, 2024
Merged

Add infrastructure to support SARIF output #701

merged 7 commits into from
Sep 3, 2024

Conversation

frabert
Copy link
Contributor

@frabert frabert commented Aug 12, 2024

No description provided.

@frabert frabert force-pushed the sarif branch 2 times, most recently from 9308479 to 476eeba Compare August 12, 2024 15:48
@trailofbits trailofbits deleted a comment from github-actions bot Aug 12, 2024
@trailofbits trailofbits deleted a comment from github-actions bot Aug 26, 2024
@frabert frabert force-pushed the sarif branch 3 times, most recently from 1cd3fc5 to 0cf1e64 Compare August 26, 2024 11:37
@trailofbits trailofbits deleted a comment from github-actions bot Aug 26, 2024
@trailofbits trailofbits deleted a comment from github-actions bot Aug 27, 2024
@frabert frabert force-pushed the sarif branch 8 times, most recently from 86bb3e2 to 1facda2 Compare August 29, 2024 12:09
@frabert frabert changed the title cc: emit diagnostics as SARIF report Add infrastructure to support SARIF output Aug 29, 2024
@frabert frabert marked this pull request as ready for review August 29, 2024 12:23
@frabert frabert requested a review from xlauko as a code owner August 29, 2024 12:23
xlauko
xlauko reviewed Aug 30, 2024
View reviewed changes
.github/workflows/build.yml Outdated Show resolved Hide resolved
include/vast/Analysis/Sarif/Sarif.hpp Outdated Show resolved Hide resolved
@frabert frabert force-pushed the sarif branch 3 times, most recently from 9ef3166 to 5f0e95f Compare August 30, 2024 10:21
@frabert frabert requested a review from xlauko August 30, 2024 10:34
xlauko
xlauko reviewed Aug 30, 2024
View reviewed changes
test/lit.site.cfg.py.in Outdated Show resolved Hide resolved
xlauko
xlauko reviewed Aug 30, 2024
View reviewed changes
include/vast/Analysis/Sarif/Sarif.hpp Outdated Show resolved Hide resolved
include/vast/Analysis/Sarif/Sarif.hpp Outdated Show resolved Hide resolved
lib/vast/Frontend/Consumer.cpp Outdated Show resolved Hide resolved
lib/vast/Frontend/Consumer.cpp Outdated Show resolved Hide resolved
test/vast/Compile/Sarif/argc.c Show resolved Hide resolved
@frabert frabert force-pushed the sarif branch 4 times, most recently from c7fad6a to efccba5 Compare August 30, 2024 13:14
@frabert frabert requested a review from xlauko August 30, 2024 13:28
xlauko
xlauko reviewed Sep 3, 2024
View reviewed changes
include/vast/Analysis/Sarif/Sarif.hpp Outdated Show resolved Hide resolved
include/vast/Analysis/Sarif/Sarif.hpp Outdated Show resolved Hide resolved
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Sep 3, 2024

Cpp-Linter Report ⚠️

Some files did not pass the configured checks!

clang-format reports: 3 file(s) not formatted
  • include/vast/Frontend/Sarif.hpp
  • lib/vast/Frontend/Consumer.cpp
  • lib/vast/Frontend/Sarif.cpp

Have any feedback or feature suggestions? Share it here.

@xlauko xlauko merged commit 613d397 into master Sep 3, 2024
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pgoodman pgoodman pgoodman left review comments

@xlauko xlauko xlauko approved these changes

Assignees

@frabert frabert

Labels
analysis
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@frabert @pgoodman @xlauko