Integrations

[CraigHarley]

@danhper I'd like to help with this if I can, (send coverage status on pull requests). Opening a PR for visibility.

[danhper]

Hi and thanks for proposing to help.

So there are quite a few things to do to get this to work.
GitHub is deprecating services to move to what they call "GitHub apps" so I think this would be the way to go.

• Link GitHub account
• List GitHub project
  A list of GitHub projects should be shown when clicking on the Add from GitHub. I think it would probably involve calling this API: https://developer.github.com/v3/apps/installations/#list-repositories-accessible-to-the-user-for-an-installation
• Install app to GitHub repository
  When the project is added, we need to install OpenCov to the GitHub repository. I suppose it would be through this endpoint: https://developer.github.com/v3/apps/installations/#add-repository-to-installation and will require the write permission on checks
• Create a "check" on GitHub when the coverage is processed.
  This will be done through this API https://developer.github.com/v3/checks/runs/#create-a-check-run I think it would be good to start by simply showing the change in the coverage and we can later on think about configure the build to fail if the coverage drops by more than X or these kind of things.

I guess this should be enough to get a first working version.
It would be awesome if you had time to give it a shot. I would of course be more than happy to review any PR.

Thanks.

[relistan]

@danhper first thanks for the project, it's working great for us. Re: this PR, you don't need most of that UI, it's provided by GitHub. "All" you need to do is respond to a couple of endpoints and be able to trigger runs.

[danhper]

@relistan I'm happy to know that it is working for you.
Thanks for the comment.
I do not have much time to work on this now but help would definitely be welcome.

[yknx4]

https://github.com/yknx4/librecov
I've added support for both PR comments and commit checks in my form in case you are still interested

[danhper]

@yknx4 Nice!! If you'd like to send a PR I'd be happy to merge it in!