Peter Solnica

Results 573 comments of Peter Solnica

Feel free to open a draft PR early if you'd like to get some feedback sooner.

@jameskbride hey πŸ˜„ this is on hold but *I think* @ianwhite is still interested in working on this, so please try to sync up with him first.

@anmacagno I think it's perfectly fine to do it like that for the time being, but eventually we definitely want a dedicated feature that would make it easier

@tbsvttr ah thanks, I know it's long-time coming but we'll get to implement it eventually πŸ™‚

@jacobtani not really :( currently it doesn't seem like there's anybody who'd have time to work on this feature. Like I mentioned, we'll get to this eventually. It's on my...

This is a change in Rails that broke your code, by using auto-inject you define a constructor and it looks like it's no longer compatible with Rails 7. dry-auto_inject shouldn't...

This was moved from dry-auto_inject because we could come up with a nice integration with the controller API. Marking it as help-wanted as I don't have time (for now at...

> I thought dry-auto-_inject is supported the ruby ecosystem, including rails. I should clarify - `dry-auto_inject` works with Rails but you assume you can use it with any class, which...

@flash-gordon do we want to do this initially in rom-sql and then maybe add some stuff to core, or start with core and extend it in rom-sql?

> But I'm not sure that should we really add it to dry-container. @davydovanton why?