script/default-site: read Jekyll source from local clone #5188

Merged
merged 1 commit into from Aug 2, 2016

Conversation

Projects
None yet
3 participants
@parkr
Member

parkr commented Aug 2, 2016

If you bump the version in the repo, then it all goes haywire.

Fixes build failures in #5187.

script/default-site: read Jekyll source from local clone
If you bump the version in the repo, then it all goes haywire.

@parkr parkr referenced this pull request Aug 2, 2016

Merged

Site updates for v3.2.1 #5187

@parkr parkr added the tests label Aug 2, 2016

@mattr-

This comment has been minimized.

Show comment
Hide comment
@mattr-

mattr- Aug 2, 2016

Member

LGTM

Member

mattr- commented Aug 2, 2016

LGTM

@parkr

This comment has been minimized.

Show comment
Hide comment
@parkr

parkr Aug 2, 2016

Member

LGTM.

Member

parkr commented Aug 2, 2016

LGTM.

@parkr

This comment has been minimized.

Show comment
Hide comment
@parkr

parkr Aug 2, 2016

Member

That's the first time I have had 2 LGTM's in a while on a PR here! Need to work on building out the @jekyll/core team a little so we can have more 👀 , or expand who can give LGTM's.

@jekyllbot: merge +dev

Member

parkr commented Aug 2, 2016

That's the first time I have had 2 LGTM's in a while on a PR here! Need to work on building out the @jekyll/core team a little so we can have more 👀 , or expand who can give LGTM's.

@jekyllbot: merge +dev

@jekyllbot jekyllbot merged commit a7bc7ff into master Aug 2, 2016

2 of 3 checks passed

continuous-integration/appveyor/pr AppVeyor build failed
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
jekyll/lgtm Approved by @mattr- and @parkr.

@jekyllbot jekyllbot deleted the default-site-read-from-local branch Aug 2, 2016

jekyllbot added a commit that referenced this pull request Aug 2, 2016

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