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
Rename issue #3039 #3048
Rename issue #3039 #3048
Conversation
Rename DISABLE_INDEXES_PLUGIN_* settings #3039 |
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.
DISABLE_INDEXES_PLUGIN_INDEX_AND_ATOM_FEED
is still mentioned in docs/creating-a-site.rst and docs/sphinx/creating-a-site.rst
AUTHORS.txt
Outdated
@@ -1,4 +1,4 @@ | |||
* `Agustin Henze <https://github.com/agustinhenze>`_ | |||
* `Agustin Henze <https://github.com/agustinhenze>`_ |
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.
You inserted a UTF-8 BOM here (a hexdump shows that the file now starts with ef bb bf
).
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.
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.
I fixed this for now. In the future, please make sure your editor does not insert BOMs (never use notepad.exe to edit stuff like that)
Signed-off-by: Chris Warrick <kwpolska@gmail.com>
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 you please add warnings for both options, like we do on nikola.py line 714 for UNSLUGIFY_TITLES
?
nikola/nikola.py
Outdated
self.config['DISABLE_INDEXES'] = self.config['DISABLE_INDEXES_PLUGIN_INDEX_AND_ATOM_FEED'] | ||
|
||
if 'DISABLE_INDEXES_PLUGIN_RSS_FEED' in self.config: | ||
utils.LOGGER.warn('The DISABLE_INDEXES_PLUGIN_RSS_FEED setting was renamed DISABLE_MAIN_RSS_FEED') |
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.
You forgot one line here, but I’m going to add it myself.
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.
LGTM!
Thanks for contributing, @Knapstad! |
@Kwpolska thank you for letting me :) and thanks for your patiens, its my first contribution out side my own projects |
Pull Request Checklist
Description