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

Rework .travis.yml #186

Merged
merged 3 commits into from
Apr 15, 2020
Merged

Rework .travis.yml #186

merged 3 commits into from
Apr 15, 2020

Conversation

bufferoverflow
Copy link
Member

@bufferoverflow bufferoverflow commented Apr 8, 2020

No description provided.

@coveralls
Copy link

coveralls commented Apr 8, 2020

Coverage Status

Coverage remained the same at 100.0% when pulling 235b951 on fix/jruby-build into 715cc44 on master.

@bufferoverflow bufferoverflow changed the title Update .travis.yml WIP: Update .travis.yml Apr 8, 2020
@bufferoverflow bufferoverflow changed the title WIP: Update .travis.yml Update .travis.yml Apr 8, 2020
@bufferoverflow bufferoverflow changed the title Update .travis.yml Rework .travis.yml Apr 9, 2020
Copy link
Member

@toupeira toupeira left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bufferoverflow LGTM! 👍

Not sure about the removal of the appraisal gem, maybe we want to keep it around in case we'll need it again in the future? On the other hand it won't be much effort to add it back, so I really don't have a strong opinion on this 🙂

@bufferoverflow
Copy link
Member Author

Thanks @toupeira I think we can add the appraisal any time if needed. I prefer to have only the things we really need.

@bufferoverflow
Copy link
Member Author

@md5 @supernova32 Could you review?

@bufferoverflow bufferoverflow merged commit 3e055c6 into master Apr 15, 2020
@bufferoverflow bufferoverflow deleted the fix/jruby-build branch April 15, 2020 22:06
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

3 participants