Fix for Rails 7.1 -- rename to run_with

[mathieujobin]

Because rails 7.1 recently introduce Object#with

https://github.com/rails/rails/commits/main/activesupport/lib/active_support/core_ext/object/with.rb

fixes #132

[leandromoreira]

Do we still need to offer with? Is there any chance of breaking something?

        def with
          yield self
        end

        def run_with
          yield self
        end

[mathieujobin]

its broken if we keep it. I think it should be just renamed. but the name might not be your favorite..

[leandromoreira]

Thank you @mathieujobin

[leandromoreira]

lemurheavy/coveralls-ruby#163

[mathieujobin]

@leandromoreira how is that coveralls-ruby PR related to this?

[leandromoreira]

ohh @mathieujobin I was just linking the issue to give visibility to coveralls people. :)

[mathieujobin]

I see, I should have thought about it ! Thanks for the merge.

Looking forward to the imminent Rails 7.1 release

[DmytroVasin]

Will this fix be backported to the 1.x version? Are there any intentions to that?

[mathieujobin]

@DmytroVasin is there any reason to remain on 1.x with rails 7.1 ?

[DmytroVasin]

@mathieujobin
In our project, we sucked with 1.x version because that is the last version that depends on redis (not redis-client)

But I assume that is my problem :)
Thank you!

P.S. Sorry for the long response :(