Skip to content
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

[READY] Update API documentation #911

Merged
merged 1 commit into from
Jan 25, 2018
Merged

Conversation

micbou
Copy link
Collaborator

@micbou micbou commented Jan 24, 2018

The website containing the API docs wasn't updated when merging PR #857 (the /receive_messages handler is missing). This PR fixes some typos, renames openapi.yaml to openapi.yml, and updates the website.


This change is Reviewable

Fix typos.
Rename the extension of the OpenAPI file to .yml.
@codecov-io
Copy link

codecov-io commented Jan 25, 2018

Codecov Report

Merging #911 into master will increase coverage by 1.08%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master    #911      +/-   ##
=========================================
+ Coverage   95.11%   96.2%   +1.08%     
=========================================
  Files          83      83              
  Lines        6279    6500     +221     
=========================================
+ Hits         5972    6253     +281     
+ Misses        307     247      -60

@Valloric
Copy link
Member

:lgtm:

Thanks for the PR!

@bstaletic
Copy link
Collaborator

:lgtm:

@zzbot r=Valloric


Reviewed 4 of 4 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved, some commit checks failed.


Comments from Reviewable

@zzbot
Copy link
Contributor

zzbot commented Jan 25, 2018

📌 Commit 6bf33e8 has been approved by Valloric

@zzbot
Copy link
Contributor

zzbot commented Jan 25, 2018

⌛ Testing commit 6bf33e8 with merge e1181df...

zzbot added a commit that referenced this pull request Jan 25, 2018
[READY] Update API documentation

The website containing the API docs wasn't updated when merging PR #857 (the `/receive_messages` handler is missing). This PR fixes some typos, renames `openapi.yaml` to `openapi.yml`, and updates the website.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/valloric/ycmd/911)
<!-- Reviewable:end -->
@zzbot
Copy link
Contributor

zzbot commented Jan 25, 2018

💔 Test failed - status-appveyor

@bstaletic
Copy link
Collaborator

@zzbot retry

@zzbot
Copy link
Contributor

zzbot commented Jan 25, 2018

⌛ Testing commit 6bf33e8 with merge 9a022ee...

zzbot added a commit that referenced this pull request Jan 25, 2018
[READY] Update API documentation

The website containing the API docs wasn't updated when merging PR #857 (the `/receive_messages` handler is missing). This PR fixes some typos, renames `openapi.yaml` to `openapi.yml`, and updates the website.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/valloric/ycmd/911)
<!-- Reviewable:end -->
@zzbot
Copy link
Contributor

zzbot commented Jan 25, 2018

💔 Test failed - status-appveyor

@micbou
Copy link
Collaborator Author

micbou commented Jan 25, 2018

@zzbot retry

@zzbot
Copy link
Contributor

zzbot commented Jan 25, 2018

⌛ Testing commit 6bf33e8 with merge 4fdf29f...

zzbot added a commit that referenced this pull request Jan 25, 2018
[READY] Update API documentation

The website containing the API docs wasn't updated when merging PR #857 (the `/receive_messages` handler is missing). This PR fixes some typos, renames `openapi.yaml` to `openapi.yml`, and updates the website.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/valloric/ycmd/911)
<!-- Reviewable:end -->
@zzbot
Copy link
Contributor

zzbot commented Jan 25, 2018

☀️ Test successful - status-appveyor, status-travis
Approved by: Valloric
Pushing 4fdf29f to master...

@zzbot zzbot merged commit 6bf33e8 into ycm-core:master Jan 25, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants