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 for Rust 1.85.0 #953

Merged
merged 1 commit into from
Feb 21, 2025
Merged

Fix for Rust 1.85.0 #953

merged 1 commit into from
Feb 21, 2025

Conversation

macpie
Copy link
Member

@macpie macpie commented Feb 20, 2025

No description provided.

@@ -34,7 +34,7 @@ where
Box::pin(
handle
.map_err(anyhow::Error::from)
.and_then(|result| async move { result.map_err(anyhow::Error::from) }),
.and_then(|result| async move { result }),
Copy link
Member

Choose a reason for hiding this comment

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

huh? this doesn't actually do anything right?

Copy link
Member Author

Choose a reason for hiding this comment

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

It does

   |
34 | /         Box::pin(
35 | |             handle.map_err(anyhow::Error::from),
36 | |             // .and_then(|result| async move { result }),
37 | |         )
   | |_________^ expected `Result<(), Error>`, found `Result<Result<(), Error>, Error>`
   |
   = note: expected enum `std::result::Result<(), _>`
              found enum `std::result::Result<std::result::Result<(), anyhow::Error>, _>````

@macpie macpie merged commit a276699 into main Feb 21, 2025
17 checks passed
@macpie macpie deleted the macpie/rust-1.85.0 branch February 21, 2025 19:48
michaeldjeffrey pushed a commit that referenced this pull request Feb 25, 2025
michaeldjeffrey pushed a commit that referenced this pull request Feb 25, 2025
# 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.

5 participants