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
Add pretty option #8
Conversation
Thanks for the PR! Since |
Yes of course, will do it asap |
Ok, done! |
@@ -17,4 +17,5 @@ generateSitemap(process.stdout, { | |||
findRoot: argv._[0] || '.', | |||
ignoreFile: argv['ignore-file'], | |||
prefix: prefix, | |||
pretty: argv.pretty, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could this use the minimist opts to avoid needing --pretty=true
and allowing just --pretty
? Would look like:
var argv = require('minimist')(process.argv.slice(2), { 'boolean': 'pretty' })
Yeah nice, added that |
👍 Thanks! |
Hey, another PR! My site uses pretty URLs, so
www.example.com/author/index.html
becomeswww.example.com/author
. This PR adds a--pretty
option to removeindex.html
from the output. It also means thatwww.example.com/about.html
becomeswww.example.com/about
, because a lot of static site frameworks (e.g. Netlify) feature this kind of URL rewrite.