Skip to content

Commit 19aadb2

Browse files
GH-46662: [CI][Dev] Fix shellcheck SC2148 errors in ci/scripts directory (#46663)
### Rationale for this change We are trying to implement shellcheck on all sh files in #44748. ### What changes are included in this PR? * SC2148: Add `shellcheck shell=bash` in `util_*` files ``` In ci/scripts/util_enable_core_dumps.sh line 1: # Licensed to the Apache Software Foundation (ASF) under one ^-- SC2148 (error): Tips depend on target shell and yours is unknown. Add a shebang or a 'shell' directive. ``` ### Are these changes tested? Yes. ### Are there any user-facing changes? No. * GitHub Issue: #46662 Authored-by: Hiroyuki Sato <[email protected]> Signed-off-by: Sutou Kouhei <[email protected]>
1 parent ea0e701 commit 19aadb2

File tree

3 files changed

+6
-0
lines changed

3 files changed

+6
-0
lines changed

.pre-commit-config.yaml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -299,7 +299,9 @@ repos:
299299
?^ci/scripts/release_test\.sh$|
300300
?^ci/scripts/ruby_test\.sh$|
301301
?^ci/scripts/rust_build\.sh$|
302+
?^ci/scripts/util_enable_core_dumps\.sh$|
302303
?^ci/scripts/util_free_space\.sh$|
304+
?^ci/scripts/util_log\.sh$|
303305
?^cpp/build-support/build-lz4-lib\.sh$|
304306
?^cpp/build-support/build-zstd-lib\.sh$|
305307
?^cpp/build-support/get-upstream-commit\.sh$|

ci/scripts/util_enable_core_dumps.sh

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
1+
# shellcheck shell=bash
2+
#
13
# Licensed to the Apache Software Foundation (ASF) under one
24
# or more contributor license agreements. See the NOTICE file
35
# distributed with this work for additional information

ci/scripts/util_log.sh

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,5 @@
1+
# shellcheck shell=bash
2+
#
13
# Licensed to the Apache Software Foundation (ASF) under one
24
# or more contributor license agreements. See the NOTICE file
35
# distributed with this work for additional information

0 commit comments

Comments
 (0)