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

Test with modern Ruby release #13

Merged
merged 1 commit into from Oct 9, 2019
Merged

Conversation

mjgiarlo
Copy link
Contributor

@mjgiarlo mjgiarlo commented Oct 9, 2019

This commit allows the build to go green again and includes a few changes:

  • Simplify the test matrix by only testing Ruby 2.5.3 (for now). Needs discussion by maintainers.
  • Fixnums are now Integers
  • Allow bundler 1.x or 2.x

This commit allows the build to go green again and includes a few changes:
* Simplify the test matrix by only testing Ruby 2.5.3 (for now). Needs discussion by maintainers.
* Fixnums are now Integers
* Allow bundler 1.x or 2.x
@mjgiarlo
Copy link
Contributor Author

mjgiarlo commented Oct 9, 2019

@alexduryee The build passes now: https://travis-ci.org/alexduryee/timetwister/builds/595676034 Not sure why GitHub is not showing the checks above? Maybe the Travis/GitHub integration needs to be tweaked/updated.

@alexduryee
Copy link
Owner

I've no strong feelings about dropping Travis support for older Rubies, so this is fine by me. Dunno why the CI badge is listing an error; it seems like it's a problem with the 1.9.3 build, so dropping that should clear it up.

@alexduryee alexduryee merged commit 20268f0 into alexduryee:master Oct 9, 2019
@mjgiarlo mjgiarlo deleted the updates branch October 9, 2019 18:18
@mjgiarlo
Copy link
Contributor Author

mjgiarlo commented Oct 9, 2019

@alexduryee It looks like this repo is no longer integrated with Travis, so the build failure up above is from a way old build. I can work on this if you are OK with granting me access to this repo. I'm comfy doing whatever here, so you know.

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.

None yet

2 participants