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

Ruby 3.0 Support & CI Actions improvement #124

Merged
merged 4 commits into from
Jan 14, 2021

Conversation

jsantos
Copy link
Contributor

@jsantos jsantos commented Dec 28, 2020

- Relax Ruby version constraint with `'>= 2.4', '< 3.1'`;
- CI now runs with multiple versions of Ruby, using a matrix strategy;
  - Currently runs on 2.6, 2.7 and 3.0 (previously was only running on 2.6)
- Replaced the `bundle install` step with `bundler-cache: true`
  - As shown on https://github.com/ruby/setup-ruby#caching-bundle-install-automatically
@codecov
Copy link

codecov bot commented Dec 28, 2020

Codecov Report

Merging #124 (2013b36) into master (04bcb6f) will increase coverage by 1.42%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #124      +/-   ##
==========================================
+ Coverage   90.52%   91.95%   +1.42%     
==========================================
  Files           1        1              
  Lines         359      348      -11     
==========================================
- Hits          325      320       -5     
+ Misses         34       28       -6     
Impacted Files Coverage Δ
lib/codecov.rb 91.95% <0.00%> (+1.42%) ⬆️

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 04bcb6f...2013b36. Read the comment docs.

Copy link
Contributor

@AlexWayfer AlexWayfer left a comment

Choose a reason for hiding this comment

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

Thank you, especially for matrix in GitHub Actions (do we still need in Travis CI?), but I have a few questions and suggestions.

.github/workflows/ruby.yml Outdated Show resolved Hide resolved
codecov.gemspec Outdated Show resolved Hide resolved
jsantos and others added 2 commits December 28, 2020 21:21
Co-authored-by: Alexander Popov <alex.wayfer@gmail.com>
Co-authored-by: Alexander Popov <alex.wayfer@gmail.com>
@jsantos
Copy link
Contributor Author

jsantos commented Dec 28, 2020

Thank you, especially for matrix in GitHub Actions (do we still need in Travis CI?), but I have a few questions and suggestions.

I was initially editing the Travis CI file and got curious about which CI system was being used. Then I noticed GH Actions and started focusing on that. Guess if it's confirmed as not being used, travis.yml can be removed!

@AlexWayfer
Copy link
Contributor

AlexWayfer commented Dec 28, 2020

Guess if it's confirmed as not being used, travis.yml can be removed!

Even if so (we should wait for a response from maintainers), I think it should be made in a separate PR. I've just raised this question for discussion when noticed changes.

@n-rodriguez
Copy link

n-rodriguez commented Dec 30, 2020 via email

@ShockwaveNN
Copy link

Sooo, any news on merging it?

Also, as a side note that this github actions can be improved via testing head version of ruby, but allowing this to fail

something like this:

    runs-on: ubuntu-latest
    continue-on-error: ${{ matrix.allow-failures }}
    strategy:
      matrix:
        ruby: [2.4, 2.5, 2.6, 2.7, 3.0]
        allow-failures: [false]
        include:
          - ruby: head
            allow-failures: true

So next time in December ruby 3.1 will be ready to release - CI can be sure that new version is working fine

@AlexWayfer
Copy link
Contributor

Also, as a side note that this github actions can be improved via testing head version of ruby, but allowing this to fail

Honestly, I often see such practice, but don't think that it's really helpful and matter. CI bloating (imagine an impact on all PRs during this year), but new versions has incompatible very rarely. Maintainers or contributors can manually test gems with RC versions if they want to ensure that everything is OK, or they know about related changes.

@ShockwaveNN
Copy link

@thomasrockhu any news on merging it or adding support of ruby 3.0 in other way?
Or is there some particular problem with this PR and something holding it back?

Co-authored-by: Alexander Popov <alex.wayfer@gmail.com>
@thomasrockhu
Copy link
Contributor

Thanks for the contribution! I'll look into releasing tomorrow or early next week.

@thomasrockhu thomasrockhu merged commit a8c4914 into codecov:master Jan 14, 2021
@jsantos jsantos deleted the ruby3_support branch January 15, 2021 01:44
@ahmetabdi
Copy link

Thanks @thomasrockhu

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.

None yet

6 participants