Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix a typo in axum::extract::rejection::FailedToDeserializeQueryString documentation #2009

Merged
merged 1 commit into from
May 19, 2023

Conversation

j3rrywan9
Copy link
Contributor

Motivation

Fix a typo in public documentation.

Solution

Fix a typo in public documentation.

Copy link
Member

@davidpdrsn davidpdrsn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@davidpdrsn davidpdrsn enabled auto-merge (squash) May 18, 2023 08:24
@j3rrywan9
Copy link
Contributor Author

@davidpdrsn Thanks for the review! Do you happen to know how to go about fixing the failing cargo-hack check?

@davidpdrsn
Copy link
Member

Not sure but it's unrelated to your patch. I'll look into it separately.

@j3rrywan9
Copy link
Contributor Author

@davidpdrsn I created a separate PR #2013 to fix the failing cargo hack check.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
…ng` documentation
auto-merge was automatically disabled May 18, 2023 18:49

Head branch was pushed to by a user without write access

@j3rrywan9
Copy link
Contributor Author

@davidpdrsn Can you please approve it again? I did a force push after rebase.

@davidpdrsn davidpdrsn merged commit 59d6a7b into tokio-rs:main May 19, 2023
@j3rrywan9 j3rrywan9 deleted the patch-1 branch May 19, 2023 08:38
davidpdrsn pushed a commit that referenced this pull request Jul 16, 2023
@davidpdrsn davidpdrsn mentioned this pull request Jul 16, 2023
davidpdrsn pushed a commit that referenced this pull request Jul 17, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants