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

Move require "jekyll/drops/drop" to "jekyll.rb" Linux does not read files in alphanumeric order, this can lead to Jekyll drops not working on Linux because the assumption here is that the collection drop will be required first. #4292

Merged
merged 1 commit into from Dec 27, 2015

Conversation

Projects
None yet
3 participants
@jekyllbot
Contributor

jekyllbot commented Dec 27, 2015

PR automatically created for @envygeeks.

Move require "jekyll/drops/drop" to "jekyll.rb"

Linux does not read files in alphanumeric order, this can lead to
Jekyll drops not working on Linux because the assumption here is that
the collection drop will be required first.```
Move require "jekyll/drops/drop" to "jekyll.rb"
Linux does not read files in alphanumeric order, this can lead to
Jekyll drops not working on Linux because the assumption here is that
the collection drop will be required first.
@parkr

This comment has been minimized.

Show comment
Hide comment
@parkr

parkr Dec 27, 2015

Member

:shipit: once Travis passes.

Member

parkr commented Dec 27, 2015

:shipit: once Travis passes.

envygeeks added a commit that referenced this pull request Dec 27, 2015

Merge pull request #4292 from jekyll/pull/fix-liquid-drop-requires
Move require "jekyll/drops/drop" to "jekyll.rb"

Linux does not read files in alphanumeric order, this can lead to
Jekyll drops not working on Linux because the assumption here is that
the collection drop will be required first.

@envygeeks envygeeks merged commit 43618c9 into master Dec 27, 2015

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

@envygeeks envygeeks deleted the pull/fix-liquid-drop-requires branch Dec 27, 2015

@jekyll jekyll locked and limited conversation to collaborators Feb 27, 2017

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.