search

ManageIQ / inventory_refresh

Apache License 2.0
1 stars 23 forks source link

Test with ruby 3.2 and rails 7.1 #122

Closed agrare closed 6 months ago

agrare commented 9 months ago
miq-bot commented 6 months ago

This pull request has been automatically marked as stale because it has not been updated for at least 3 months.

If these changes are still valid, please remove the stale label, make any changes requested by reviewers (if any), and ensure that this issue is being looked at by the assigned/reviewer(s).

agrare commented 6 months ago

ruby 3.2 was failing because of an old version of rake (~> 10.0) being required

kbrock commented 6 months ago

@agrare Do you want to update the code climate code, as well?

Maybe ruby version 3.0?

if: ${{
  github.ref == 'refs/heads/master' &&
  matrix.ruby-version == '2.7' &&
  matrix.rails-version == '6.1'
}}
agrare commented 6 months ago

Do you want to update the code climate code, as well?

:+1: good call, updated

miq-bot commented 6 months ago

Checked commit https://github.com/agrare/inventory_refresh/commit/565a66da14f2d567263177a8322951ae806f92b0 with ruby 2.7.8, rubocop 1.56.3, haml-lint 0.51.0, and yamllint 0 files checked, 0 offenses detected Everything looks fine. :cake: