Move msgpack dep to benchmark group in Gemfile #16

Merged
merged 4 commits into from Jan 4, 2017

Projects

None yet

2 participants

@brianmario
Owner

The msgpack gem isn't required for anything other than benchmarking so let's skip building it in all of our automated scripts.

Should fix the build errors we were seeing in #15.

cc @kivikakk

@brianmario move msgpack dep to gemfile
f807494
@kivikakk kivikakk added a commit that referenced this pull request Jan 4, 2017
@kivikakk kivikakk Port #16 to v1-release 820310e
kivikakk added some commits Jan 4, 2017
@kivikakk kivikakk Specify install step
6aa6be5
@kivikakk kivikakk Get latest bundler to avoid 1.9.3 error
4296302
@kivikakk kivikakk Avoid 2.1.2 error
fc91dda
@kivikakk kivikakk merged commit ced75e5 into master Jan 4, 2017

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
@kivikakk kivikakk deleted the brianmario/fix-msgpack-dep branch Jan 4, 2017
@brianmario
Owner

🙇

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