Skip to content
This repository has been archived by the owner on Feb 27, 2023. It is now read-only.

docs: auto generate Dragonfly cli/api/contributors docs via code #1151

Merged
merged 1 commit into from
Jan 2, 2020

Conversation

pouchrobot
Copy link
Collaborator

Signed-off-by: pouchrobot pouch-dev@alibaba-inc.com

1.Describe what this PR did
This PR is automatically done by AI-based collaborating robot.
Pouchrobot will auto-generate cli/api document via https://github.com/spf13/cobra/tree/master/doc every day.

2.Does this pull request fix one issue?
None

3.Describe how you did it
We use the following user input CLI document generating command in pouchrobot to generate CLI doc:
make build && make install && make docs

For API part, we use a tool swagger2markup to make it.

4.Describe how to verify it
None

5.Special notes for reviews
The cli/api doc must be automatically generated.

Signed-off-by: pouchrobot <pouch-dev@list.alibaba-inc.com>
@codecov-io
Copy link

Codecov Report

Merging #1151 into master will decrease coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1151      +/-   ##
==========================================
- Coverage   47.24%   47.23%   -0.02%     
==========================================
  Files         116      116              
  Lines        7217     7217              
==========================================
- Hits         3410     3409       -1     
  Misses       3540     3540              
- Partials      267      268       +1
Impacted Files Coverage Δ
supernode/daemon/mgr/scheduler/manager.go 21.91% <0%> (-0.69%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 509945a...2d8dec6. Read the comment docs.

@allencloud
Copy link
Contributor

Please help check if the API doc change satisfies the corresponding pull request #1065 @yunfeiyanggzq . Thanks a lot.

@yunfeiyanggzq
Copy link
Member

Please help check if the API doc change satisfies the corresponding pull request #1065 @yunfeiyanggzq . Thanks a lot.

the swagger.yml only add "/api/v1" prefix before verrsion 1.0 url in #1065,this auto generated api doc does right.the doc changes a lot because the order is not the same as before @allencloud

@allencloud
Copy link
Contributor

LGTM

@allencloud allencloud merged commit 9aaea24 into dragonflyoss:master Jan 2, 2020
@pouchrobot pouchrobot added the LGTM one maintainer or community participant agrees to merge the pull reuqest. label Jan 2, 2020
sungjunyoung pushed a commit to sungjunyoung/Dragonfly that referenced this pull request May 8, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
areas/docs LGTM one maintainer or community participant agrees to merge the pull reuqest. size/XL
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants