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

Ruby and Rails matrix in CI #398

Merged
merged 12 commits into from
Aug 24, 2017

Conversation

RobinDaugherty
Copy link
Member

@RobinDaugherty RobinDaugherty commented Aug 17, 2017

A gemfile for each set of dependencies that Better Errors intends to support. These are:

  • Rails 4.2, 5.0, and 5.1, without binding_of_caller or haml
  • Rails 4.2, 5.0, and 5.1, with haml
  • Rails 4.2, 5.0, and 5.1, with binding_of_caller
  • Pure Rack, with and without binding_of_caller
  • Pry 0.9, 0.10, (and 0.11pre, which is currently failing tests)

Added these to Travis config, matrixed with each major version of MRI. Turns out that, for example, "2.4" does not mean "the latest 2.4", so I had to change the Ruby versions in Travis to the latest minor release of each.

At this time, Pry 0.10 does not pass specs. This is a real issue with Pry 0.10 that will need to be resolved. It's in the allowed_failures because of this.

and move build-specific dependencies to the Appraisals.
It seems travis doesn't use the latest of each version specified, even though rvm does. rvm/rvm#3512 (comment)
We have to customize nokogiri version based on ruby version, and Appraisal doesn't support that. We'll have to maintain the gemfiles manually.
@RobinDaugherty RobinDaugherty changed the title Test matrix Ruby and Rails matrix in CI Aug 24, 2017
@RobinDaugherty RobinDaugherty merged commit e9a352d into BetterErrors:master Aug 24, 2017
@RobinDaugherty RobinDaugherty deleted the feature/test-matrix branch August 24, 2017 02:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant