Skip to content
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

Update to axum 0.6.0 #369

Merged
merged 2 commits into from
Nov 25, 2022
Merged

Update to axum 0.6.0 #369

merged 2 commits into from
Nov 25, 2022

Conversation

davidpdrsn
Copy link
Contributor

@davidpdrsn davidpdrsn commented Nov 25, 2022

Resolves #345

Copy link
Owner

@juhaku juhaku left a comment

Choose a reason for hiding this comment

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

Super 👍

@juhaku
Copy link
Owner

juhaku commented Nov 25, 2022

There is one doc test that fails. You can run tests locally for utoipa-swagger-ui e.g running script: ./scripts/tests.sh utoipa-swagger-ui.


failures:

error: doctest failed, to rerun pass `-p utoipa-swagger-ui --doc`
---- src/lib.rs - (line 93) stdout ----
error[E0599]: the function or associated item `new` exists for struct `Router<B>`, but its trait bounds were not satisfied
  --> src/lib.rs:105:25
   |
15 |  let app = Router::<B>::new()
   |                         ^^^ function or associated item cannot be called on `Router<B>` due to unsatisfied trait bounds
   |
   = note: the following trait bounds were not satisfied:
           `B: Clone`
           `B: Sync`
help: consider restricting the type parameters to satisfy the trait bounds
   |
13 |     B: HttpBody + Send + 'static, B: Clone, B: Sync
   |                                 ~~~~~~~~~~~~~~~~~~~

error: aborting due to previous error

For more information about this error, try `rustc --explain E0599`.
Couldn't compile the test.

failures:
    src/lib.rs - (line 93)

@davidpdrsn
Copy link
Contributor Author

@juhaku I believe it should be fixed. Wanna poke the CI button?

@juhaku juhaku merged commit b558b36 into juhaku:master Nov 25, 2022
@juhaku
Copy link
Owner

juhaku commented Nov 25, 2022

Sure

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Released
Development

Successfully merging this pull request may close these issues.

None yet

2 participants