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
Fix NoMethodError undefined method 'coverage' #1255
Conversation
Gemfile
Outdated
@@ -18,7 +18,7 @@ group :lint, :development do | |||
end | |||
|
|||
group :test, :development do | |||
gem 'coveralls', require: false | |||
gem 'coveralls_reborn', '~> 0.21.0', require: false |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we can allow any version of coveralls_reborn.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The fix was only introduced for v "0.21.0" but I removed the explicit version as you recommended.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for setting this up!
This is a fix for lostisland#1254 As mentioned in coveralls-ruby issue: lemurheavy/coveralls-ruby#161, using coveralls-ruby-rebon should fix the problem.
91fe134
to
85926af
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for providing the solution! Together with #1256, we should be able to get coveralls back up 🎉
@iMacTia I'm glad that this small fix was helpful. Feel free to share any issue up-for-grabs in this project and I can dedicate some free time to it. |
We now get the following
So it seems reports are now sent correctly to Coveralls 👍! |
@Maroo-b Thanks! I'll send anything that pops in my mind, but for now I'd suggest to look at the "help wanted" and "good first issue" labels in the repo issues 🎉 |
Description
This is a fix for #1254
As mentioned in coveralls-ruby issue: lemurheavy/coveralls-ruby#161, using coveralls-ruby-rebon should fix the problem.
Here is a screenshot from the CI logs as a proof :)