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

Conversation

@Kwpolska
Copy link
Member

commented Jun 23, 2015

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

Includes changes in other documentation.

Comments?

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

Fix #1827 -- human-written manpage
Signed-off-by: Chris Warrick <kwpolska@gmail.com>
Set version correctly (due to change above)
Signed-off-by: Chris Warrick <kwpolska@gmail.com>
@da2x

This comment has been minimized.

Copy link
Contributor

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

This comment has been minimized.

Copy link
Member Author

commented Jun 23, 2015

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

Improve manpage, fix typos in handbook
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

3 checks passed

continuous-integration/appveyor AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@Kwpolska Kwpolska deleted the modern-manpages branch Jun 24, 2015

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants
You can’t perform that action at this time.