Santiago Bartesaghi

Results 29 comments of Santiago Bartesaghi

I agree with @owst. I opened a PR, in case the maintainers are fine with the change we also need to change the README and probably other docs too.

@oestrich @jakehow what's the maintenance status of this gem? Are you looking for new maintainers? It'd be quite important to get this trivial PR merged 😬 Thanks 🙌

FWIW in https://github.com/rootstrap/rails_api_base we migrated to: - [RSpec OpenAPI](https://github.com/exoego/rspec-openapi) for generating API documentation - [RSwag](https://github.com/rswag/rswag) for serving the API documentation More info in https://github.com/rootstrap/rails_api_base/blob/main/docs/api_documentation.md

Hi @johnnyshields, I think that the problem you are solving is legit, thanks for your contribution 🙌 As you said, this PR adds several features, which IMO makes it hard...

Hey, just in case, no, I'm not a maintainer. >That would be possible, but since the internal Dalli/RedisProxy classes would be now raising an error (instead of suppressing it as...

@jamiemccarthy I'm sorry if I left the wrong impression 🙏 @jrochkind Don't worry, I'll make sure to always clarify I'm not a maintainer from now on 👍 What I do...

Hi, I think this is a cool feature to add 💯 Probably it's minor, but wanted to leave here my only concern that is that if someone in the future...

> Would appreciate 🙏 if someone would help me fix CI for older rubies. Seems like there are failures in every Ruby version (not just old ones) e.g. [Ruby 3.3](https://github.com/rack/rack-attack/actions/runs/8951229444/job/24587320296?pr=442)....

Hi @jamiemccarthy thanks a lot for the well detailed description. I think you brought up an interesting problem, but I'm still not clear on what the solution should be. I'm...

Thanks for clarifying @jamiemccarthy, it makes sense now. After giving it more thought, I believe that the core of your solution is the right one, and that it would be...