Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Minor improvements to 'Command Line Applications' docs #426

Merged
merged 3 commits into from
May 12, 2014

Conversation

dicato
Copy link
Contributor

@dicato dicato commented May 9, 2014

I noticed "command line" was referred to in a few different ways; "command-line" and "commandline". The latter was probably a typo. They have been changed to "command line".

Additionally, strip trailing whitespace. Hopefully that's acceptable.

dicato added 2 commits May 9, 2014 16:35
Standardize on "command line", not "command-line".
@sigmavirus24
Copy link
Contributor

They should be command-line. I have always seen the first two you reference, but never your style (with a space). Please change these to hypens (-) to be consistent.

@dicato
Copy link
Contributor Author

dicato commented May 12, 2014

You're right, should be fixed now!

I took the liberty of changing the references in the header and todo as well.

Thanks!

@sigmavirus24
Copy link
Contributor

Thanks @Locker537! 🍰

sigmavirus24 added a commit that referenced this pull request May 12, 2014
Minor improvements to 'Command Line Applications' docs
@sigmavirus24 sigmavirus24 merged commit f5eb99f into realpython:master May 12, 2014
@dicato dicato deleted the cli_fixes branch May 12, 2014 16:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants