Skip to content
This repository was archived by the owner on Jul 23, 2025. It is now read-only.

Add SARIF support #74

Merged
merged 3 commits into from
Sep 22, 2021
Merged

Add SARIF support #74

merged 3 commits into from
Sep 22, 2021

Conversation

lmt-swallow
Copy link
Member

@lmt-swallow lmt-swallow commented Sep 22, 2021

Description

This PR adds SARIF support, which can be used to integrate Shisho with a lot of code scanning platforms. This closes #65.

Checklist

  • I opened a draft PR or added the [WIP] to the title if my PR is not ready for review.
  • I have reviewed the code by myself.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have made corresponding changes to the documentation.
  • I have added tests enough to show how your code behaves and that your code works as expected.

Additional Notes

N/A

@netlify
Copy link

netlify bot commented Sep 22, 2021

✔️ Deploy Preview for cranky-fermi-e6c846 canceled.

🔨 Explore the source changes: af8508f

🔍 Inspect the deploy log: https://app.netlify.com/sites/cranky-fermi-e6c846/deploys/614b506f30ec6f0008380dc4

# This is the 1st commit message:

wip: add initial implementation to support SARIF

# This is the commit message #2:

wip: fix tedious bugs

Signed-off-by: Takashi Yoneuchi <takashi.yoneuchi@shift-js.info>

# This is the commit message #3:

bugfix: fix duplication of rules

Signed-off-by: Takashi Yoneuchi <takashi.yoneuchi@shift-js.info>

# This is the commit message #4:

bugfix: fix path info

Signed-off-by: Takashi Yoneuchi <takashi.yoneuchi@shift-js.info>

# This is the commit message #5:

[TEST] Add a GitHub Actions workflow with SARIF feature  (#73)

Signed-off-by: Takashi Yoneuchi <takashi.yoneuchi@shift-js.info>
# This is the commit message #6:

feat: support multiple patterns & rewrite patterns (#75)

Signed-off-by: Takashi Yoneuchi <takashi.yoneuchi@shift-js.info>
wip: fix tedious bugs

Signed-off-by: Takashi Yoneuchi <takashi.yoneuchi@shift-js.info>

bugfix: fix duplication of rules

Signed-off-by: Takashi Yoneuchi <takashi.yoneuchi@shift-js.info>

bugfix: fix path info

Signed-off-by: Takashi Yoneuchi <takashi.yoneuchi@shift-js.info>

[TEST] Add a GitHub Actions workflow with SARIF feature  (#73)

Signed-off-by: Takashi Yoneuchi <takashi.yoneuchi@shift-js.info>

feat: support multiple patterns & rewrite patterns (#75)

Signed-off-by: Takashi Yoneuchi <takashi.yoneuchi@shift-js.info>

bugfix: enhance diff views (#76)

Signed-off-by: Takashi Yoneuchi <takashi.yoneuchi@shift-js.info>

wip: resolve conflicts

Signed-off-by: Takashi Yoneuchi <takashi.yoneuchi@shift-js.info>

chore: tidy up the test workflow

Signed-off-by: Takashi Yoneuchi <takashi.yoneuchi@shift-js.info>
@lmt-swallow lmt-swallow merged commit 571184c into main Sep 22, 2021
@lmt-swallow lmt-swallow deleted the feature/sarif-support branch September 22, 2021 15:53
@lmt-swallow lmt-swallow mentioned this pull request Sep 23, 2021
5 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Better Integration with GitHub
1 participant