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

Fix #2628 - allow meta generator to be diabled #2634

Merged
merged 1 commit into from Jan 14, 2017

Conversation

adarshdec23
Copy link
Contributor

No description provided.

Copy link
Contributor

@felixfontein felixfontein left a 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)
Copy link
Contributor

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.

Copy link
Member

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.

@Kwpolska Kwpolska dismissed felixfontein’s stale review January 14, 2017 09:35

This is so minor that I’ll fix it myself, so no need to withhold this PR.

@Kwpolska Kwpolska merged commit a9656c9 into getnikola:master Jan 14, 2017
@felixfontein
Copy link
Contributor

Sure, fine for me :)

@felixfontein
Copy link
Contributor

Thanks @adarshdec23 for implementing this!

@Kwpolska
Copy link
Member

Thanks for contributing to Nikola, @adarshdec23!

Kwpolska added a commit that referenced this pull request Jan 14, 2017
Signed-off-by: Chris Warrick <kwpolska@gmail.com>
@adarshdec23
Copy link
Contributor Author

Thanks for the merge! :-)

felixfontein added a commit that referenced this pull request Jan 14, 2017
felixfontein added a commit that referenced this pull request Jan 14, 2017
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.

None yet

3 participants