Skip to content

Conversation

@mshick
Copy link

@mshick mshick commented May 12, 2016

The following adds support for Jekyll 3 and removes some deprecation warnings.

@XhmikosR
Copy link
Contributor

@parkr: can you review and merge this?

Not sure all the patches in this PR are needed. I only saw the docs.reverse warning with Jekyll 3.

@parkr
Copy link
Member

parkr commented Aug 9, 2016

Looks like Travis is having a tough time with our current configuration, and octopress-multilingual is causing issues as well. Oh, fun.


if categories = page.data['paginate']['categories']
collection = collection.reject{|p| (p.categories & categories).empty?}
collection = collection.reject{|p| (p.data['categories'] & categories).empty?}
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@XhmikosR I think these p.data changes are still required.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can it be because I don't use categories so that code isn't run at all?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That would make sense, yes.

@XhmikosR
Copy link
Contributor

@parkr: do you need any help with this? I can try rebasing the branch.

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.

4 participants