-
Notifications
You must be signed in to change notification settings - Fork 283
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
tower: prepare to release v0.4.5 #558
Conversation
Mind if we also try to get #557 in? Should be merged shortly. |
Sure thing. I'll be going to bed in ~1 hour so feel free to take over this PR. |
tower/CHANGELOG.md
Outdated
|
||
### Fixed | ||
|
||
- **util**: Make combinators implement `Debug` in more cases. ([#552]) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: can we clarify "in more cases"? specifically, I think this is that the Debug
impl doesn't require the closure/function to be Debug
...and in practice, i think these types will never implement Debug
....
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
tower/CHANGELOG.md
Outdated
- **util**: Add `option_layer` to convert an `Option<Layer>` into a `Layer`. ([#555]) | ||
- **builder**: Add `ServiceBuilder::option_layer` to optionally add a layer. ([#555]) | ||
- **steer**: `Steer` now implements `Clone | ||
- **make**: Added `Shared` which lets you implement `MakeService` by cloning a |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: the tense here is inconsistent with the other entries ("Added ..." rather than "Add ..."). We should probably try to keep that consistent across releases, but let's at least keep it consistent here?
(it's entirely possible that i'm the only person in the world who cares about this, sorry!)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Its all about the little things 😊
@davidpdrsn if you don't mind rebasing onto master to pick up #557, I think we'll be good to go! |
3aec71a
to
c7a5d96
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lovely! looks good to merge pending CI
No description provided.