securesauce / precaution-beta

Precaution provides a simple, automated code review for GitHub projects by running code linters with a security focus on pull requests.
Other
1 stars 0 forks source link

Create check runs per linter #237

Open ericwb opened 5 years ago

ericwb commented 5 years ago

Is your feature request related to a problem? Please describe. So today, we have a single Check run named Precaution that runs multiple linters within it.

A better model might be to utilize GitHubs support to run multiple "check runs" per app.
Each run would have it's own status and could be re-run individually in case of error. Each check run could also be customized to be appropriate for the linter in use. Additional, the runs would run in parallel.

Describe the solution you'd like One check suite named Precaution with check runs:

Describe alternatives you've considered n/a

Additional context If you look at other GitHub apps, they make use of multiple runs.

ericwb commented 5 years ago

This would also mean we could drop the merge_reports.js we are currently using.