Skip to content

Conversation

randycoulman
Copy link

When running the latest version of Ink with Jekyll 3.x, several deprecation warnings are shown. This fixes the deprecation warnings by following the instructions given.

This fix will break compatibility with Jekyll 2. It looks like some of the other gems in the Octopress family are doing that, but if it is necessary to remain compatible with both Jekyll 2 and 3, this fix will need to be re-worked somehow. I'm not sure of the best way to do that, so suggestions are most welcome.

There might be a similar issue in octopress-multilingual, but I haven't used that gem, so can't say for sure.

Fixes #64.

The `merged_file_read_opts` method is in the `Jekyll::Utils` module and
takes a `site` argument.

Note that this is the same fix as in octopress#65; I just wanted to have a
functional version of ink for Jekyll 3.
@dgmstuart
Copy link

I reckon this should be a breaking change.

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.

Jekyll 3 changed site.posts from an Array to a Collection which causes deprecation warning in Ink
2 participants