From 2c28558c98a3d7b25db4f9c25c1400b0b024d85d Mon Sep 17 00:00:00 2001 From: Andrew Lamb Date: Mon, 16 Dec 2024 16:30:47 -0500 Subject: [PATCH] Revert "Minor: make it easier to find instructions when fmt fails" This reverts commit efe8af4e19a2a68e1b0f662f5d97aeb3cd35e0af. --- .github/workflows/rust.yml | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/.github/workflows/rust.yml b/.github/workflows/rust.yml index 22c95e5075b..ff5040fd294 100644 --- a/.github/workflows/rust.yml +++ b/.github/workflows/rust.yml @@ -101,13 +101,12 @@ jobs: - name: Format arrow run: cargo fmt --all -- --check - name: Format parquet - # Many modules in parquet are skipped, so check parquet separately + # Many modules in parquet are skipped, so check parquet separately. If this check fails, run: + # cargo fmt -p parquet -- --config skip_children=true `find ./parquet -name "*.rs" \! -name format.rs` + # from the top level arrow-rs directory and check in the result. # https://github.com/apache/arrow-rs/issues/6179 working-directory: parquet - run: | - # if this fails, run this from the parquet directory: - # cargo fmt -p parquet -- --config skip_children=true `find ./parquet -name "*.rs" \! -name format.rs` - cargo fmt -p parquet -- --check --config skip_children=true `find . -name "*.rs" \! -name format.rs` + run: cargo fmt -p parquet -- --check --config skip_children=true `find . -name "*.rs" \! -name format.rs` - name: Format object_store working-directory: object_store run: cargo fmt --all -- --check