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

Prepare version 5.2.0 #478

Merged
merged 2 commits into from
Dec 19, 2017
Merged

Conversation

mikwat
Copy link
Contributor

@mikwat mikwat commented Dec 14, 2017

There have been a lot of changes since 5.1.0 There was an attempt at this back in 2016: #405 Hoping to see it through this time.

@wldcordeiro

@coveralls
Copy link

coveralls commented Dec 14, 2017

Coverage Status

Coverage remained the same at 93.089% when pulling d2602a5 on mikwat:prepare-5.2.0 into bf6b9a1 on RolifyCommunity:master.

@wldcordeiro
Copy link
Member

@mikwat CI seems to fail. I'd love to get some help from some more active Ruby developers in getting this to work with Rails 5 and only the other supported versions.

@mikwat
Copy link
Contributor Author

mikwat commented Dec 14, 2017

@wldcordeiro Strange that CI passed on master 24 days ago... let me try to figure out what's going on.

@coveralls
Copy link

coveralls commented Dec 14, 2017

Coverage Status

Coverage remained the same at 93.089% when pulling bf611a2 on mikwat:prepare-5.2.0 into bf6b9a1 on RolifyCommunity:master.

@mikwat
Copy link
Contributor Author

mikwat commented Dec 14, 2017

@wldcordeiro CI passing now!

@mikwat
Copy link
Contributor Author

mikwat commented Dec 15, 2017

Sorry to bug you @wldcordeiro, but I'd really like to move this forward. Please take another look.

Copy link
Member

@wldcordeiro wldcordeiro left a comment

Choose a reason for hiding this comment

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

Thanks a ton @mikwat I had to get rolify setup on my laptop again to get the rubygems stuff prepped but I'll bring this in and release. 😄

@wldcordeiro wldcordeiro merged commit 154748c into RolifyCommunity:master Dec 19, 2017
@mikwat mikwat deleted the prepare-5.2.0 branch December 19, 2017 19:26
@mikwat
Copy link
Contributor Author

mikwat commented Dec 19, 2017

Great, thanks @wldcordeiro!

@mikwat mikwat mentioned this pull request Dec 21, 2017
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.

3 participants