-
Notifications
You must be signed in to change notification settings - Fork 28
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
Add implementation guide #22
Conversation
Codecov Report
@@ Coverage Diff @@
## master #22 +/- ##
=========================================
Coverage 90.87% 90.87%
Complexity 69 69
=========================================
Files 11 11
Lines 274 274
Branches 6 6
=========================================
Hits 249 249
Misses 23 23
Partials 2 2 Continue to review full report at Codecov.
|
docs/implementation-guide.md
Outdated
@@ -0,0 +1,33 @@ | |||
# Implementation Guide | |||
|
|||
This document guides you how to implement this checks API for a specific remote SCM platform. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This document guides you how to implement this checks API for a specific remote SCM platform. | |
This document will show you how to implement the checks API for a specific SCM platform. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
some code examples might be good
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm apart from the minor nits I added suggestions on
Add mock server test for GitHub check run publishing
No description provided.