Fixed api-endpoints Purpose capitalization; added tests for Purpose and Doc for all commands #904

Merged
merged 1 commit into from Oct 14, 2014

Conversation

Projects
None yet
2 participants
@dimitern
Contributor

dimitern commented Oct 12, 2014

The "api-endpoints" command's Purpose doc lowercased.
Added a test to validate the format of all commands'
Purpose and Doc fields (proper capitalization and makes
Purpose required). Only "api-endpoints" is affected by
the test so far.

@dimitern

This comment has been minimized.

Show comment
Hide comment
@dimitern

This comment has been minimized.

Show comment
Hide comment
@dimitern

dimitern Oct 14, 2014

Contributor

$$merge$$

Contributor

dimitern commented Oct 14, 2014

$$merge$$

@dimitern

This comment has been minimized.

Show comment
Hide comment
@dimitern

dimitern Oct 14, 2014

Contributor

fixes-1380521

Contributor

dimitern commented Oct 14, 2014

fixes-1380521

@jujubot

This comment has been minimized.

Show comment
Hide comment
@jujubot

jujubot Oct 14, 2014

Contributor

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju

Contributor

jujubot commented Oct 14, 2014

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju

jujubot added a commit that referenced this pull request Oct 14, 2014

Merge pull request #904 from dimitern/api-endpoints-fix-help
Fixed api-endpoints Purpose capitalization; added tests for Purpose and Doc for all commands

The "api-endpoints" command's Purpose doc lowercased.
Added a test to validate the format of all commands'
Purpose and Doc fields (proper capitalization and makes
Purpose required). Only "api-endpoints" is affected by
the test so far.

@jujubot jujubot merged commit 2fd1d4a into juju:master Oct 14, 2014

@dimitern dimitern deleted the dimitern:api-endpoints-fix-help branch Oct 14, 2014

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