Skip to content

[CI][Dev] Fix shellcheck SC2086 and SC2223 errors in ci/scripts directory #46526

Closed

Description

@hiroyuki-sato

Describe the enhancement requested

This is the sub issue #44748.

SC2086 and SC2223 check require quoting like ${arrow_dir} -> "${arrow_dir}".

In ci/scripts/integration_arrow_build.sh line 25:
: ${ARROW_INTEGRATION_CPP:=ON}
  ^--------------------------^ SC2223 (info): This default assignment may cause DoS due to globbing. Quote it.


In ci/scripts/integration_arrow_build.sh line 29:
. ${arrow_dir}/ci/scripts/util_log.sh
  ^----------^ SC2086 (info): Double quote to prevent globbing and word splitting.

Component(s)

Continuous Integration

Activity

  1. added a commit that references this issue on May 28, 2025
  2. added this to the 21.0.0 milestone on May 28, 2025
  3. kou commented on May 28, 2025

    @kou
    Member

    Issue resolved by pull request 46527
    #46527

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions