Thanks for this action! Wanted to use it and it made me initially think that the action is not working properly 😅
Only then I noticed that sample's versioning is outdated. Fixed links to workflows as well as they were pointing to **/workflow instead of **/workflows.
Pull request checklist
Please check if your PR fulfills the following requirements:
[ ] ~Tests for the changes have been added (for bug fixes / features)~
[x] Docs have been reviewed and added / updated if needed (for bug fixes / features)
[ ] ~pre-commit was run locally and any changes were pushed~
[ ] ~test/test.sh has passed locally and any fixes were made for failures~
Pull request type
Please check the type of change your PR introduces:
[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
Hey @tomwillis608 I really like it so far! Currently looking into two additional features:
clearing up previous comments after updates to PR have been made or the workflow has re-run - can be solved by scripting on my side though
combine multiple SARIF files from multiple tools to have a single comment with all reports combined - don't know if SARIF supports having multiple tools in a single file (still new to it) - if so then I could just work around it with a script
Pull Request
Thanks for this action! Wanted to use it and it made me initially think that the action is not working properly 😅 Only then I noticed that sample's versioning is outdated. Fixed links to workflows as well as they were pointing to
**/workflow
instead of**/workflows
.Pull request checklist
Please check if your PR fulfills the following requirements:
Pull request type
Please check the type of change your PR introduces:
Does this introduce a breaking change?