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}"),
         }
     }
 

Reply via email to