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
Making path handlers of indexes plugin work before get_tasks() is run. #2047
Conversation
site.register_path_handler('index', self.index_path) | ||
site.register_path_handler('index_atom', self.index_atom_path) | ||
site.register_path_handler('section_index', self.index_section_path) | ||
site.register_path_handler('section_index_atom', self.index_section_atom_path) | ||
return super(Indexes, self).set_site(site) | ||
|
||
def _get_filtered_posts(self, lang, show_untranslated_posts): |
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.
this function name should be more informative. Also, using _underscores is not a valid reason to skip documenting things.
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.
Ah, that pep257
tool doesn't complain about missing docstrings for underscore names apparently. Good to know.
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.
It believes they are private, which often makes sense, but it’s good to throw in a docstring nevertheless.
The failed check is due to some invariance breaking, which seems to not be the fault of this PR. |
@Aeyoun, mind reviewing and merging? |
Anyone minds if I merge? |
Making path handlers of indexes plugin work before get_tasks() is run.
Thanks! |
Fixes #2040.