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

Use each instead of each_with_index in source_file.erb #53

Merged

Conversation

koic
Copy link
Contributor

@koic koic commented May 17, 2017

The behavior of Enumerable#each_with_index has changed in JRuby 9.1.9.0.

jruby/jruby#4610

This PR used each method because it do not use a block variable of index.

The purpose of this change is to fix the following error at simplecov.

simplecov-ruby/simplecov#582

Thanks.

@PragTob
Copy link
Collaborator

PragTob commented May 17, 2017

I should have checked my notifications so much earlier... that fix is of course better than mine in #54 when each_with_index is just used like each.

Thank you.

Closes #54 (does this work here?)

@PragTob PragTob merged commit 8b1479b into simplecov-ruby:master May 17, 2017
@PragTob PragTob mentioned this pull request May 17, 2017
@koic koic deleted the use_each_instead_of_each_with_index branch May 18, 2017 00:23
@koic
Copy link
Contributor Author

koic commented May 18, 2017

Thank you for review and simplecov-html 0.10.1 release. It seems that the problem has been resolved and working. Thanks again!

jsonn pushed a commit to jsonn/pkgsrc that referenced this pull request Jun 5, 2017
0.10.1 2017-05-17
========

## Bugfixes

* circumvent a regression that happens in the new JRuby 9.1.9.0 release. See [#53](simplecov-ruby/simplecov-html#53) thanks @koic
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.

2 participants