Skip to content

uniform format and layout#29389

Closed
tim-zju wants to merge 3 commits intomoby:masterfrom
tim-zju:my_dev
Closed

uniform format and layout#29389
tim-zju wants to merge 3 commits intomoby:masterfrom
tim-zju:my_dev

Conversation

@tim-zju
Copy link
Contributor

@tim-zju tim-zju commented Dec 14, 2016

- What I did
uniform format and layout
- How I did it

- How to verify it

- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

Signed-off-by: tim-zju <[email protected]>
Signed-off-by: tim-zju <[email protected]>
Copy link
Member

@AkihiroSuda AkihiroSuda left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This fix won't have any effect on rendered HTML, so I am not sure this fix is useful

@thaJeztah
Copy link
Member

I'm not sure either if we should do this; it doesn't change the output, and may lead to issues with cherry-picking changes to other versions of the docs; also, starting with docker 1.13, the API docs will be generated from the swagger.yml file.

@thaJeztah
Copy link
Member

I'm closing this PR because of the above, but thanks for contributing @tim-zju! Sorry we didn't accept your change

@thaJeztah thaJeztah closed this Dec 17, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants