/juju

Update help text per Doc Team recommendations #5022

Merged
merged 2 commits into from Apr 8, 2016

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@reedobrien @jujubot
@reedobrien
Contributor

reedobrien commented Apr 7, 2016

See details in https://goo.gl/MuP5SO

Also, note this commit removes a test for a lower case first letter in
the summary. Since they are all changing to upper case we can put this
back once done. Otherwise the tests will fail on merge and we won't be
able to build.

(Review request: http://reviews.vapour.ws/r/4464/)

reedobrien added some commits Apr 7, 2016

@reedobrien
Update help text per Doc Team recommendations 
See details in https://goo.gl/MuP5SO

Also, note this commit removes a test for a lower case first letter in
the summary. Since they are all changing to upper case we can put this
back once done. Otherwise the tests will fail on merge and we won't be
able to build.
977ca3b
@reedobrien
Minor udates to keep the backticks like `cherylj`
5cd9784
@reedobrien
Contributor

reedobrien commented Apr 8, 2016

http://reviews.vapour.ws/r/4464/

$$merge$$
@jujubot
Contributor

jujubot commented Apr 8, 2016

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

@jujubot jujubot merged commit 10fa1a5 into juju:master Apr 8, 2016

@reedobrien reedobrien deleted the reedobrien:juju-core/+bug/1555248 branch Apr 8, 2016

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