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

First take on categories, has to be used togehter with a liquid-rust patch #196

Closed
wants to merge 1 commit into from
Closed

Conversation

uwearzt
Copy link
Contributor

@uwearzt uwearzt commented Feb 28, 2017

Not meant for merge, just a first review round.

Please review the tests fixtures/posts_in_subfolder, especially categories.liquid if the idea off this pull request looks OK.

Also check if it is OK to add preserve_order to yaml-rust.

I will do the cleanup stuff after that.

TODO:

  • yaml-rust preserve_order (for test and also as a requirement)
  • Make code Rust like
  • Clean up names

…patch

TODO:

   * yaml-rust preserve_order (for test and also as a requirement)
   * Make code Rust like
   * Clean up names
@tglman
Copy link
Contributor

tglman commented Jun 11, 2017

hi,

Any updates to this PR ?

Regards

@uwearzt
Copy link
Contributor Author

uwearzt commented Jun 12, 2017

Hi tglman,

sorry, I didn't have time to work on this in the last weeks. I will try to get it done in July.

Greetings
Uwe

@uwearzt uwearzt closed this Jul 5, 2017
@uwearzt uwearzt deleted the categories branch July 5, 2017 06:55
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.

2 participants