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 #2628 - allow meta generator to be diabled #2634
Conversation
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 (except placement of change). What's missing is a call to scripts/jinjify.py
to convert the mako themes to jinja2, but we can also do that when merging.
@@ -11,6 +11,8 @@ Features | |||
* Sort posts in the following order (most important last): source path | |||
(A-Z), title (A-Z), date (reverse chronological order), priority | |||
meta number (descending). (Issue #2627) | |||
* Add META_GENERATOR_TAG option in conf.py allowing the meta generator | |||
to be disabled if needed. (Issue #2628) |
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 need to add this more at the top, before the changelist for version v7.8.3 starts. Now it looks like this was added in v7.8.3, while it wasn't.
Just start an empty Features
section at the top of the file and move the comment there.
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.
@felixfontein This should be the entire New in master/Features/Bugfixes template. I’ll take care of it myself, it’s a pretty minor thing.
This is so minor that I’ll fix it myself, so no need to withhold this PR.
Sure, fine for me :) |
Thanks @adarshdec23 for implementing this! |
Thanks for contributing to Nikola, @adarshdec23! |
Signed-off-by: Chris Warrick <kwpolska@gmail.com>
Thanks for the merge! :-) |
Still missing from #2634.
No description provided.