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

docs: update api html file #1345

Merged
merged 1 commit into from Nov 27, 2018

Conversation

Projects
None yet
3 participants
@disksing
Member

disksing commented Nov 27, 2018

What problem does this PR solve?

Update the API html dodument based on latest api.rami.

What is changed and how it works?

docker pull mattjtodd/raml2html:latest
docker run --rm -v $PWD:/raml mattjtodd/raml2html -i /raml/server/api/api.raml -o /raml/docs/api.html

Check List

Tests

  • No code

@disksing disksing requested review from nolouch and rleungx Nov 27, 2018

@gregwebs

This comment has been minimized.

Contributor

gregwebs commented Nov 27, 2018

Can you commit that docker code under hack/ ? Note that go projects seem to be moving from calling the directory hack/ to scripts/

@disksing

This comment has been minimized.

Member

disksing commented Nov 27, 2018

@gregwebs Thanks for reminding. I'll update it too.

@disksing

This comment has been minimized.

Member

disksing commented Nov 27, 2018

@gregwebs I'm going to do the adjustment on master branch. release-2.1 will be left as it is.

@disksing disksing referenced this pull request Nov 27, 2018

Merged

update pd api html file #773

@disksing disksing merged commit d12940c into pingcap:release-2.1 Nov 27, 2018

5 checks passed

ci/circleci Your tests passed on CircleCI!
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
jenkins-ci-pd/build Jenkins job succeeded.
Details
jenkins-ci-pd/unit-test Jenkins job succeeded.
Details
license/cla Contributor License Agreement is signed.
Details

@disksing disksing deleted the disksing:api-doc branch Nov 27, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment