We need to use `bundle exec jekyll` in `script/proof` #2610

Merged
merged 2 commits into from Jul 16, 2014

Conversation

Projects
None yet
3 participants
@alfredxing
Member

alfredxing commented Jul 16, 2014

Because the CI builds use Bundler to "install" Jekyll from the source dir, the jekyll command doesn't actually exist. Instead, we need to use bundle exec to build the site when proofing.

@parkr

This comment has been minimized.

Show comment
Hide comment
@parkr

parkr Jul 16, 2014

Member

Cool.

Can you also add set -e at the top of this script and script/cibuild?

Member

parkr commented Jul 16, 2014

Cool.

Can you also add set -e at the top of this script and script/cibuild?

Add `set -e` to top of `proof` and `cibuild`
Exit immediately on any non-zero exit code

@parkr parkr merged commit c4ae4ae into jekyll:master Jul 16, 2014

1 check was pending

continuous-integration/travis-ci The Travis CI build is in progress
Details

@alfredxing alfredxing deleted the alfredxing:proof-use-bundle-exec branch Jul 16, 2014

parkr added a commit that referenced this pull request Jul 16, 2014

@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.