GitList

Browse code

Fix help test comment to align with the code

I disagree with #14546 that pushed the help text past 80 chars.
Aside from it now making the help text look ugly on 80 char displays,
which I use, one thing I like about the previous limitation is that it
forced us to keep our options down to more reasonable phrases/words.
For example, I think
` --disable-content-trust=true`
could have been:
` --disable-trust=true`
or even:
` --disable-ctrust=true`

But regardless, let's at least make the comments match what the code does.

Signed-off-by: Doug Davis <dug@us.ibm.com>

Doug Davis authored on 2015/10/07 21:09:46
Showing 1 changed files

integration-cli/docker_cli_help_test.go

History View file @ 73a6ac0

@@ -76,7 +76,7 @@ func (s *DockerSuite) TestHelpTextVerify(c *check.C) {
 			}
 		}
 
-		// Make sure each cmd's help text fits within 80 chars and that
+		// Make sure each cmd's help text fits within 90 chars and that
 		// on non-windows system we use ~ when possible (to shorten things).
 		// Pull the list of commands from the "Commands:" section of docker help
 		helpCmd = exec.Command(dockerBinary, "help")

...	...	@@ -76,7 +76,7 @@ func (s DockerSuite) TestHelpTextVerify(c check.C) {
76	76	}
77	77	}
78	78
79		- // Make sure each cmd's help text fits within 80 chars and that
	79	+ // Make sure each cmd's help text fits within 90 chars and that
80	80	// on non-windows system we use ~ when possible (to shorten things).
81	81	// Pull the list of commands from the "Commands:" section of docker help
82	82	helpCmd = exec.Command(dockerBinary, "help")