Skip to content

ci: fix clippy warn error to long issue#88

Merged
JingsongLi merged 1 commit intoapache:mainfrom
luoyuxia:fix-clippy-warning-issue
Feb 12, 2026
Merged

ci: fix clippy warn error to long issue#88
JingsongLi merged 1 commit intoapache:mainfrom
luoyuxia:fix-clippy-warning-issue

Conversation

@luoyuxia
Copy link
Contributor

@luoyuxia luoyuxia commented Feb 8, 2026

Fix issue #86

I'm going to contribute back some features to paimon-rust, but the ci won't work now. So, submit a pr to fix the ci as well as running cargo clippy --all-targets --fix --allow-dirty --allow-staged do some format.

message: String,
source: opendal::Error,
#[snafu(source(from(opendal::Error, Box::new)))]
source: Box<opendal::Error>,
Copy link
Contributor Author

Choose a reason for hiding this comment

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

see https://rust-lang.github.io/rust-clippy/master/index.html#result_large_err
which suggest us to wrap big size error to Box.

@luoyuxia luoyuxia force-pushed the fix-clippy-warning-issue branch from 718125e to 63817c0 Compare February 8, 2026 11:33
@luoyuxia
Copy link
Contributor Author

luoyuxia commented Feb 8, 2026

cc @JingsongLi

@JingsongLi
Copy link
Contributor

+1 Thanks @luoyuxia

@JingsongLi JingsongLi merged commit 3ef9234 into apache:main Feb 12, 2026
7 checks passed
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.

2 participants