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

Jekyll compatibility ? #2

Closed
taophp opened this issue Feb 29, 2016 · 4 comments
Closed

Jekyll compatibility ? #2

taophp opened this issue Feb 29, 2016 · 4 comments

Comments

@taophp
Copy link
Contributor

taophp commented Feb 29, 2016

As I use markdown first to publish my texts using Jekyll, my markdown files are builds the Jekyll way. So they have a YALM header. At least, I expect Crowbook to ignore them, so I would not have to manage two versions of each text. But this is not the case, and the result is ugly.

Suggestion: it should be even better if the YAML header could be use by crowbook instead of an external configuration file. Maybe a YAML key named crowbook should be use to avoid conflict of key naming between Jekyll and Crowbook. So the file header should look like:


---
layout: post
title:  Disparition
date: 2016-01-18 00:30:05
categories: blog
type: nouvelle
crowbook:
  title: Disparition
  author: Stéphane Mourey
  lang: fr

---
And now, for your pleasure, a brand new history....

Suggestion 2: there should be some redundant keys, like the title one in my sample. So, may crowbook read the whole YAML to get them, instead of repeating them in the file. I don't know, just thinking.

@crowdagger
Copy link
Owner

Indeed, it would be a good idea. Might need to work a bit on how Crowbook's option reading currently works to allow to set options this way, but in the mean time ignoring these blocks should be relatively easy. Thanks for your input.

@crowdagger
Copy link
Owner

Partially fixed (YAML blocks should now be ignored, but it's not yet possible to set options this way)

@taophp
Copy link
Contributor Author

taophp commented Mar 1, 2016

That's great! I can manage a separate configuration file. Please, consider Crowbook configuration via YAML header of MD files as a new feature.

@taophp
Copy link
Contributor Author

taophp commented Mar 1, 2016

Wonderful! Thanks! 💯

crowdagger pushed a commit that referenced this issue Mar 15, 2016
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

No branches or pull requests

2 participants