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 a file blacklist for header permalinks #2767

Merged
merged 1 commit into from May 15, 2017

Conversation

@Kwpolska
Copy link
Member

@Kwpolska Kwpolska commented May 15, 2017

Header permalinks are not desirable in some files (eg. output/index.html on getnikola.com — which I just rebuilt using header_permalinks; deduplication is pending until it’s merged). So, here’s a blacklist.

Signed-off-by: Chris Warrick <kwpolska@gmail.com>
@Kwpolska Kwpolska added this to the v7.8.6 milestone May 15, 2017
@Kwpolska Kwpolska requested review from felixfontein and ralsina May 15, 2017
@ralsina
Copy link
Member

@ralsina ralsina commented May 15, 2017

How about a no-permalinks metadata also?

@Kwpolska
Copy link
Member Author

@Kwpolska Kwpolska commented May 15, 2017

That’s doable, but it would require mapping output locations to posts — do we have anything like that?

@ralsina
Copy link
Member

@ralsina ralsina commented May 15, 2017

@Kwpolska
Copy link
Member Author

@Kwpolska Kwpolska commented May 15, 2017

Merging as-is then.

@Kwpolska Kwpolska merged commit 1d03f5f into master May 15, 2017
5 checks passed
@Kwpolska Kwpolska deleted the header_permalinks_file_blacklist branch May 15, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants