-
-
Notifications
You must be signed in to change notification settings - Fork 14k
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
sarif-tools: init at 2.0.0 #283208
sarif-tools: init at 2.0.0 #283208
Conversation
36c9252
to
c0e76ff
Compare
#282347 is merged, can you rebase this PR against the latest nixpkgs? That should fix the tests. |
cc57f6d
to
1a896e1
Compare
@zebreus I rebased and added the changelog link, should be ready for review now :) |
1a896e1
to
5dbc873
Compare
Thanks for the review @AndersonTorres! I applied the changes, should be ready for merging now. |
Result of 4 packages built:
|
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.
Seems to work!
Co-authored-by: Anderson Torres <torres.anderson.85@protonmail.com>
Description of changes
Add https://github.com/microsoft/sarif-tools, a collection of tools for working with Static Analysis Results Interchange Format (SARIF) files.
This is a draft because it requires #282347 to fix the failing tests of the
python-docx
dependency (#282180). I made suresarif-tools
itself is fine by skippingpython-docx
's tests.Things done
nix.conf
? (See Nix manual)sandbox = relaxed
sandbox = true
nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD"
. Note: all changes have to be committed, also see nixpkgs-review usage./result/bin/
)Add a 👍 reaction to pull requests you find important.