Skip to content
This repository has been archived by the owner on Jan 10, 2024. It is now read-only.

Create check runs per linter #237

Open
ericwb opened this issue May 24, 2019 · 1 comment
Open

Create check runs per linter #237

ericwb opened this issue May 24, 2019 · 1 comment
Labels
enhancement New feature or request task-size-large
Milestone

Comments

@ericwb
Copy link
Collaborator

ericwb commented May 24, 2019

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:

  • Bandit
  • Gosec
  • tslint

Describe alternatives you've considered
n/a

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

@ericwb ericwb added the enhancement New feature or request label May 24, 2019
@ericwb
Copy link
Collaborator Author

ericwb commented May 24, 2019

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

@MVrachev MVrachev added this to the Near Future milestone Jun 14, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request task-size-large
Projects
None yet
Development

No branches or pull requests

2 participants