-
Notifications
You must be signed in to change notification settings - Fork 481
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
chore!: fix typo customed should be customized #4847
Conversation
Signed-off-by: tison <wander4096@gmail.com>
Thank you for this PR. I will delay merging it for now since it introduces breaking changes. Perhaps we can also update the reqsign version at the same time. |
@Xuanwo would you work on a reqsign fix? Or I may handle it. |
I would appreciate it if you could submit a PR to reqsign. |
Here it is - Xuanwo/reqsign#452 |
Hi, I want to address #4850 first before start developing of 0.48. So this PR will be hold until then. |
@@ -266,7 +266,7 @@ sha1 = { version = "0.10.6", optional = true } | |||
sha2 = { version = "0.10", optional = true } | |||
|
|||
# For http based services. | |||
reqsign = { version = "0.15.2", default-features = false, optional = true } | |||
reqsign = { version = "0.16", default-features = false, optional = true } |
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.
This should be fine now @Xuanwo
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.
Let's go.
@Xuanwo There is a similar typo in reqsign also. Please think of if we'd introduce a breaking change to fix it.