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

Avoid installing unecessary gems for site testing #5272

Merged
merged 1 commit into from Aug 23, 2016

Conversation

Projects
None yet
4 participants
@Anthony-Gaudino
Contributor

Anthony-Gaudino commented Aug 22, 2016

Many of the gems installed by just running bundle install are not necessary to test the Jekyll site. Currently 33 unnecessary gems are installed on my system.

This change comes with the cost of a longer command that users must copy/paste or type.

Many of the gems installed bu just running `bundle install` are not n…
…ecessary to test the Jekyll site. Changed the install command to avoid installing unecessary gems.
@parkr

This comment has been minimized.

Show comment
Hide comment
@parkr

parkr Aug 22, 2016

Member

LGTM!

Member

parkr commented Aug 22, 2016

LGTM!

@parkr parkr added the documentation label Aug 22, 2016

@envygeeks

This comment has been minimized.

Show comment
Hide comment
@envygeeks

envygeeks Aug 23, 2016

Contributor

LGTM.

Contributor

envygeeks commented Aug 23, 2016

LGTM.

@parkr

This comment has been minimized.

Show comment
Hide comment
@parkr

parkr Aug 23, 2016

Member

@jekyllbot: merge +dev

Member

parkr commented Aug 23, 2016

@jekyllbot: merge +dev

@jekyllbot jekyllbot merged commit d759cfb into jekyll:master Aug 23, 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 @parkr and @envygeeks.

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

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