-
Notifications
You must be signed in to change notification settings - Fork 65
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
Support for the GitHub Checks API #2311
Comments
GitHub Apps integration is required before that: #2312 |
@FeodorFitsner Even thought #2312 is closed, as far as I can see, this feature is not ready yet (tasks don't appear): Am I right? 😉 |
Correct, checks are not supported yet, but GItHub Apps support made it much closer. Cannot say specific ETA now. |
Why does AppVeyor push people into changing from the old style of integration to the new App when it's not feature complete? |
Because GitHub Apps is the "feature" by itself which allows user giving AppVeyor access to only hand-picked organizations/repositories rather than all repos GitHub user has access to. Also, GitHub Apps act as a "bot" rather than a user authorized the access through OAuth. |
Sorry ignore my comment. I misunderstood the problem thinking that the web hook didn't even work. |
Actually this is still a problem. When I switched from OAuth to Github App, there is no longer a webhook url displayed on the project settings, so there is no way to set up the hook without switching back to OAuth |
With GitHub Apps there is a centralized webhook managed by app vendor. You don't need a project-specific webhook anymore. |
Any updates on this? Blog post said "in the following weeks" but there has been nothing for nearing a year. We are considering moving to azure pipelines (mainly due to the better integration) but the implementation of checks support may weigh against this decision. Thanks! |
Almost one year later again and still no news? |
https://blog.github.com/2018-05-07-introducing-checks-api/
https://developer.github.com/v3/checks/
Example PR with this from Travis CI: https://github.com/pmarsceill/patrickmarsceill.com/pull/30/checks
The text was updated successfully, but these errors were encountered: