Skip to content
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

Jump to bottom

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

Merged
merged 2 commits into from Jul 16, 2014
Merged

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

merged 2 commits into from Jul 16, 2014

Conversation

alfredxing
Copy link
Member

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
Copy link
Member

parkr commented Jul 16, 2014

Cool.

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

@alfredxing
Add set -e to top of proof and cibuild
8e3ad08 
Exit immediately on any non-zero exit code
parkr added a commit that referenced this pull request Jul 16, 2014
@parkr
Merge pull request #2610 from alfredxing/proof-use-bundle-exec
c4ae4ae
@parkr parkr merged commit c4ae4ae into jekyll:master Jul 16, 2014
@alfredxing alfredxing deleted the proof-use-bundle-exec branch July 16, 2014 20:14
parkr added a commit that referenced this pull request Jul 16, 2014
@parkr
Update history to reflect merge of #2610 [ci skip]
2a1e7cb
@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.
Reviewers
No reviews
Assignees
No one assigned
Labels
frozen-due-to-age
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@alfredxing @parkr @jekyllbot