Join GitHub today
GitHub is home to over 50 million developers working together to host and review code, manage projects, and build software together.
Sign upGitHub is where the world builds software
Millions of developers and companies build, ship, and maintain their software on GitHub — the largest and most advanced development platform in the world.
Cosmetic changes to script #55
Conversation
Ticket: LOGGLY-4252
Ticket: LOGGLY-4104
|
Need this pulled ASAP. |
| @@ -976,7 +976,7 @@ def modify_syslog_config_file(syslog_id, syslog_configuration_details, | |||
|
|
|||
| if len(syslog_configuration_details.get("token")) <= 0: | |||
| question = ("\nThe Loggly configuration will be appended to (%s) file." | |||
| "\n\nDo you want this installer to modify " | |||
| "\n\nDo you want this setup-script to modify " | |||
vinhn
Nov 21, 2013
How about: "Would you like to have the configuration file modified?"
If not, "setup script" is fine...without the dash in between.
How about: "Would you like to have the configuration file modified?"
If not, "setup script" is fine...without the dash in between.
| @@ -1427,14 +1427,14 @@ def parse_args(self, *args, **kwargs): | |||
| CMD_USAGE = ''' | |||
| %prog <action> [option] | |||
| Action: | |||
| \tinstall Configure the syslog | |||
| \tuninstall Remove changes made by the syslog configuration script | |||
| \tsetup Configure your syslog installtion | |||
vinhn
Nov 21, 2013
"installtion" is missing an "a". For consistency though, this should perhaps read as: "Configure your syslog setup"
"installtion" is missing an "a". For consistency though, this should perhaps read as: "Configure your syslog setup"
| \tinstall Configure the syslog | ||
| \tuninstall Remove changes made by the syslog configuration script | ||
| \tsetup Configure your syslog installtion | ||
| \trevert Remove changes made by the syslog configuration script |
vinhn
Nov 21, 2013
...made by this syslog configuration script.
...made by this syslog configuration script.
|
Vinh -- thanks for the proofreading. |
Ticket: LOGGLY-4252
|
With Vinh's review comments addressed, I am going to self-merge this. Due to deployed FrontEnd changes, our current script doesn't work -- so this is hot. |
Cosmetic changes to script
No description provided.