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

Adding SARIF result format support for HTMLHint #1045

Open
shaopeng-gh opened this issue May 31, 2022 · 3 comments
Open

Adding SARIF result format support for HTMLHint #1045

shaopeng-gh opened this issue May 31, 2022 · 3 comments
Labels
keep-unstale The issue will not be marked as stale by the stale-bot PR needed

Comments

@shaopeng-gh
Copy link

Hello! We are interested in adding support for output in the open-standard SARIF format to the HTMLHint. SARIF support is required to integrate it in GitHub code scanning. Doing so would make it available to every repo in GitHub and could result in increase in user base.

More about SARIF here:
What is SARIF?
Why SARIF?

If you agree, we will be happy to looking into necessary code changes and also provide guidance on creating a starter workflow to make it available as a GitHub code scanner.

I see the tool already has a --format flag so we are looking at adding sarif to that list. Let me know if have any instructions, thanks!

Thanks,
Shaopeng

@coliff
Copy link
Member

coliff commented Jun 17, 2022

heya @shaopeng-gh - thanks for reaching out about this. sounds interesting and I like the idea of having HTMLHint available as a GitHub code scanner. Please do feel free to make a PR with the necessary changes.

@stale

This comment was marked as outdated.

@stale stale bot added the bot:stale Issue marked as stale because there was no activity label Sep 20, 2022
@shaopeng-gh shaopeng-gh closed this as not planned Won't fix, can't repro, duplicate, stale Sep 20, 2022
@shaopeng-gh
Copy link
Author

sorry was occupied by another task, I am back on this and start working on it, thanks.

@shaopeng-gh shaopeng-gh reopened this Nov 1, 2022
@stale stale bot removed the bot:stale Issue marked as stale because there was no activity label Nov 1, 2022
@coliff coliff added PR needed keep-unstale The issue will not be marked as stale by the stale-bot labels Nov 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
keep-unstale The issue will not be marked as stale by the stale-bot PR needed
Projects
None yet
Development

No branches or pull requests

2 participants