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

Fix #1827 -- human-written manpage #1844

Merged
merged 3 commits into from Jun 24, 2015
Merged

Fix #1827 -- human-written manpage #1844

merged 3 commits into from Jun 24, 2015

Conversation

Kwpolska
Copy link
Member

Output: https://gist.github.com/Kwpolska/412b6b61ecf04ad39ca7

Includes changes in other documentation.

Comments?

Signed-off-by: Chris Warrick kwpolska@gmail.com

Signed-off-by: Chris Warrick <kwpolska@gmail.com>
Signed-off-by: Chris Warrick <kwpolska@gmail.com>
@da2x
Copy link
Contributor

da2x commented Jun 23, 2015

Line 10, “[--demo]”?

Line 22, why not suggest using “-b” argument?

Line 25, add something about “generated from template” and “meets industry web standards”?

Line 32, first word should be “Its” (“It’s” == “It is”).

Line 94, add short explanation that “DEPLOY_COMMANDS is where you put your scp/rsync/ftp commands to copy the output up to a public server.”

Line 106, include https://plugins.getnikola.com/?

Line 109, include https://themes.getnikola.com/?

Line 123, it’s customary to include who wrote the man page, for what version, and when at the bottom.

@Kwpolska
Copy link
Member Author

Half the document (incl. 10, 22, 25, 32) is copied from the handbook. I’ll fix both.

10: skipped that when copying, added back.
22, 32, 106, 109: done.
25: Please provide an example sentence. How should I phrase that and what buzzwords should I include?
123: is strongly discouraged

Signed-off-by: Chris Warrick <kwpolska@gmail.com>
Kwpolska added a commit that referenced this pull request Jun 24, 2015
@Kwpolska Kwpolska merged commit 82ed5cf into master Jun 24, 2015
@Kwpolska Kwpolska deleted the modern-manpages branch June 24, 2015 12:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants