Skip to content
This repository was archived by the owner on Jan 10, 2023. It is now read-only.

Fix jruby-9.1.x compatibility#79

Merged
thomasrockhu merged 4 commits intocodecov:masterfrom
mainameiz:fix-compatibility-with-jruby-9.1.12.0
Jul 27, 2020
Merged

Fix jruby-9.1.x compatibility#79
thomasrockhu merged 4 commits intocodecov:masterfrom
mainameiz:fix-compatibility-with-jruby-9.1.12.0

Conversation

@mainameiz
Copy link
Copy Markdown
Contributor

@mainameiz mainameiz commented Jul 26, 2020

While making a PR into aasm. I found that new version of codecov does not work in jruby 9.1.x (9.1.12.0 actually).
You can find failed build on travis-ci.org in the first commit in this PR.

Reason is that GzipWriter#close does not return instance of underlying IO-like object.

It was broken in 0.1.20 release here.

Changes:

  • make the code to be jruby-9.1.x compatible

First commit in this PR adds jruby 9.1.12.0 into .travis.yml to show that it fails before changes and passes after fixing.

@mainameiz mainameiz changed the title Run tests against jruby-9.1.12.0 Fix jruby-9.1.12.0 compatibility Jul 26, 2020
@codecov
Copy link
Copy Markdown

codecov Bot commented Jul 26, 2020

Codecov Report

Merging #79 into master will increase coverage by 1.59%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #79      +/-   ##
==========================================
+ Coverage   88.65%   90.24%   +1.59%     
==========================================
  Files           1        1              
  Lines         326      328       +2     
==========================================
+ Hits          289      296       +7     
+ Misses         37       32       -5     
Impacted Files Coverage Δ
lib/codecov.rb 90.24% <100.00%> (+1.59%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 627ce66...53b8751. Read the comment docs.

@mainameiz mainameiz changed the title Fix jruby-9.1.12.0 compatibility Fix jruby-9.1.x compatibility Jul 26, 2020
@mainameiz mainameiz marked this pull request as ready for review July 26, 2020 16:22
@thomasrockhu thomasrockhu self-requested a review July 27, 2020 04:49
Comment thread .travis.yml
@thomasrockhu thomasrockhu merged commit f0b24e2 into codecov:master Jul 27, 2020
@mainameiz mainameiz deleted the fix-compatibility-with-jruby-9.1.12.0 branch July 28, 2020 08:00
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants