faktory-rs icon indicating copy to clipboard operation
faktory-rs copied to clipboard

Tech debt

Open rustworthy opened this issue 9 months ago • 4 comments

Threads that needed to be addressed after the recent "async" PR merge:

  • [x] only require references from the bindings' user where we could do without owning values internally as per this discussion;
  • [x] make Client hold a boxed dyn Connection as per thread;
  • [x] a tech debt from the recent merge as per thread;
  • [ ] address https://github.com/jonhoo/faktory-rs/pull/65#discussion_r1677116648

This change is Reviewable

rustworthy avatar May 20 '24 07:05 rustworthy