pronto icon indicating copy to clipboard operation
pronto copied to clipboard

[Enhancement] Support the GitHub Checks API

Open zhuravel opened this issue 6 years ago • 9 comments

Learn more about Checks: https://developer.github.com/changes/2018-05-07-new-checks-api-public-beta/ https://help.github.com/articles/about-status-checks/#checks https://developer.github.com/v3/checks/

checks_annotation

zhuravel avatar May 10 '18 13:05 zhuravel

IMO looks very promising! I only wonder if we should wait until it leaves the beta or assume the API won't change that much and jump into implementation.

mknapik avatar May 15 '18 07:05 mknapik

Looks like its also only for GitHub Apps at this point.

The Checks API is only available to GitHub Apps through a new granular permission: checks.

Correct me if I'm wrong, but pronto still runs using an Access Token? Thus not really qualified as a GitHub App.

kevinjalbert avatar Jun 15 '18 02:06 kevinjalbert

If there were a Checks API formatter, then that could use App credentials instead of Access Tokens. I would not see Checks API replacing the existing GitHub-oriented formatters, just providing an alternative.

aergonaut avatar Sep 07 '18 22:09 aergonaut

This also makes me think we should make the formatters pluggable like the runners are. That way the community could provide a Checks API formatter without needing to have it merged to the main gem.

aergonaut avatar Sep 07 '18 22:09 aergonaut

https://github.com/haya14busa/reviewdog is a similar project that has checks API support. They have a server that receives the comments from the linter and the server communicates with the githubs Checks API.

Maybe this is a good approach. A simple server that anyone can run in a small heroku instance.

blackjid avatar Nov 23 '18 11:11 blackjid

Pluggable formatters is a really good idea.

In lieu of that, we set up a GitHub Action to run Pronto (ruby runners) and post results to the Checks API. The CheckRun API seems pretty stable. Hope this is helpful to someone!

Is there any interest in this supporting other languages or runners?

jbielick avatar Oct 08 '19 13:10 jbielick

FYI we made a Github Action for running pronto with a Check Run formatter to handle this. Github Actions are given an App token so it was a good fit.

jbielick avatar Apr 01 '20 15:04 jbielick