This is an automated email from the ASF dual-hosted git repository.
alamb pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/arrow-rs.git
The following commit(s) were added to refs/heads/main by this push:
new e8017f7cfb Minor: fix clippy after logical conflict (#7803)
e8017f7cfb is described below
commit e8017f7cfbb9b49e5402ec8c219b531f998a9d09
Author: Andrew Lamb <[email protected]>
AuthorDate: Fri Jun 27 06:43:09 2025 -0400
Minor: fix clippy after logical conflict (#7803)
# Which issue does this PR close?
We generally require a GitHub issue to be filed for all bug fixes and
enhancements and this helps us generate change logs for our releases.
You can link an issue to this PR using the GitHub syntax.
- Follow on to https://github.com/apache/arrow-rs/pull/7757
# Rationale for this change
Clippy is failing after merging
https://github.com/apache/arrow-rs/pull/7757 due to a logical conflict
and a new clippy release:
https://github.com/apache/arrow-rs/actions/runs/15924256410/job/44917809525
```
error: variables can be used directly in the `format!` string
--> parquet-variant/src/variant/list.rs:130:25
|
130 | Err(err) => panic!("validation error: {}", err),
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: for further information visit
https://rust-lang.github.io/rust-clippy/master/index.html#uninlined_format_args
= note: `-D clippy::uninlined-format-args` implied by `-D warnings`
= help: to override `-D warnings` add
`#[allow(clippy::uninlined_format_args)]`
help: change this to
|
1[30](https://github.com/apache/arrow-rs/actions/runs/15924256410/job/44917809525#step:6:31)
- Err(err) => panic!("validation error: {}", err),
130 + Err(err) => panic!("validation error: {err}"),
|
```
# What changes are included in this PR?
Fix clippy to get CI clean on main
# Are these changes tested?
By CI
# Are there any user-facing changes?
No
---
parquet-variant/src/variant/list.rs | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/parquet-variant/src/variant/list.rs
b/parquet-variant/src/variant/list.rs
index 42f97fa0d3..320cdbbee9 100644
--- a/parquet-variant/src/variant/list.rs
+++ b/parquet-variant/src/variant/list.rs
@@ -127,7 +127,7 @@ impl<'m, 'v> VariantList<'m, 'v> {
match self.try_get(index) {
Ok(variant) => Some(variant),
- Err(err) => panic!("validation error: {}", err),
+ Err(err) => panic!("validation error: {err}"),
}
}