Skip to content

clippy: io_other_error#5955

Merged
yihau merged 1 commit intoanza-xyz:masterfrom
yihau:bump-rust-nightly-2025-03-31_clippy_other_error
Apr 24, 2025
Merged

clippy: io_other_error#5955
yihau merged 1 commit intoanza-xyz:masterfrom
yihau:bump-rust-nightly-2025-03-31_clippy_other_error

Conversation

@yihau
Copy link
Copy Markdown
Member

@yihau yihau commented Apr 24, 2025

Problem

part of nightly rust bumping #5952

https://rust-lang.github.io/rust-clippy/master/index.html#io_other_error

Summary of Changes

std::io::Error::new(io::ErrorKind::Other, "xxx") => std::io::Error::other("xxx")

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Apr 24, 2025

Codecov Report

❌ Patch coverage is 19.20000% with 101 lines in your changes missing coverage. Please review.
✅ Project coverage is 82.8%. Comparing base (8d5c4ba) to head (cf2566a).
⚠️ Report is 3501 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #5955   +/-   ##
=======================================
  Coverage    82.8%    82.8%           
=======================================
  Files         830      830           
  Lines      377289   377216   -73     
=======================================
- Hits       312526   312507   -19     
+ Misses      64763    64709   -54     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@yihau yihau requested review from brooksprumo and steviez April 24, 2025 11:22
@yihau yihau marked this pull request as ready for review April 24, 2025 11:22
Copy link
Copy Markdown

@brooksprumo brooksprumo left a comment

Choose a reason for hiding this comment

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

🪖

@yihau yihau merged commit fa11e1e into anza-xyz:master Apr 24, 2025
30 checks passed
@yihau yihau deleted the bump-rust-nightly-2025-03-31_clippy_other_error branch April 24, 2025 13:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants