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

add configuration value for sitemap.xml filename #26

Merged
merged 3 commits into from
Jul 29, 2020

Conversation

SabotageAndi
Copy link
Contributor

resolves #8

I hope this is correct. This is the first python code that I wrote.

Copy link
Owner

@jdillard jdillard left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Congrats for being the first time, this is great! I just had a small suggestion to change sitemap_name to sitemap_filename so it hopefully reads clearly if someone is scanning over the conf.py config values.

sphinx_sitemap/__init__.py Outdated Show resolved Hide resolved
sphinx_sitemap/__init__.py Outdated Show resolved Hide resolved
SabotageAndi and others added 2 commits July 29, 2020 10:21
Co-authored-by: Jared Dillard <jared.dillard+github@gmail.com>
Co-authored-by: Jared Dillard <jared.dillard+github@gmail.com>
@SabotageAndi
Copy link
Contributor Author

Sure, the new suggestion is better. I simply took the suggestion from the issue.

@jdillard jdillard merged commit f50fb63 into jdillard:master Jul 29, 2020
@jdillard
Copy link
Owner

Ah, that makes sense :) It's merged now, I'll cut a release soon. Thanks again!

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.

Set config value for sitemap filename
2 participants