Closed jrafanie closed 1 year ago
@jrafanie Sorry for the late response! Can we merge this PR? I think so, but want to make sure it's ready and works fine, since this repo doesn't have automated builds.
@jrafanie Sorry for the late response! Can we merge this PR? I think so, but want to make sure it's ready and works fine, since this repo doesn't have automated builds.
@pyrooka let me rebase and test locally and report back if it's still good
@jrafanie Sorry for the late response! Can we merge this PR? I think so, but want to make sure it's ready and works fine, since this repo doesn't have automated builds.
@pyrooka let me rebase and test locally and report back if it's still good
ok, @pyrooka, I rebased and tested all of my rubies via:
bundle install
bundle exec rake
They all pass the cop checks and tests.
ruby-2.7.8
ruby-3.0.6
ruby-3.1.4
ruby-3.2.2
Let me know what we can do to get tests running on here via travis or github actions.
Thanks for checking it! I merge this now and will take a look how we could resurrect the Travis builds in this repo.
Resolves the rubocop warnings mentioned in https://github.com/IBM/ruby-sdk-core/pull/41
Note, this should probably bump the version since we're somewhat dropping support for older rubies.