hiroyuki-sato opened a new issue, #50535:
URL: https://github.com/apache/arrow/issues/50535

   ### Describe the enhancement requested
   
   This is the sub issue #44748.
   
   * SC2034: `test_dir` appears unused. Verify it or export it.
   * SC1091: Not following: (error message here)
   * SC2086: Double quote to prevent globbing and word splitting.
   
   ```
   shellcheck ci/scripts/python_test.sh
   
   In ci/scripts/python_test.sh line 23:
   test_dir=${1}/python/build/dist
   ^------^ SC2034 (warning): test_dir appears unused. Verify use (or export if 
used externally).
   
   
   In ci/scripts/python_test.sh line 26:
     . "${ARROW_PYTHON_VENV}/bin/activate"
       ^-- SC1091 (info): Not following: ./bin/activate: openBinaryFile: does 
not exist (No such file or directory)
   
   
   In ci/scripts/python_test.sh line 45:
   : ${PYARROW_TEST_ACERO:=${ARROW_ACERO:-ON}}
     ^-- SC2223 (info): This default assignment may cause DoS due to globbing. 
Quote it.
   
   
   In ci/scripts/python_test.sh line 46:
   : ${PYARROW_TEST_AZURE:=${ARROW_AZURE:-ON}}
     ^-- SC2223 (info): This default assignment may cause DoS due to globbing. 
Quote it.
   
   
   In ci/scripts/python_test.sh line 47:
   : ${PYARROW_TEST_CUDA:=${ARROW_CUDA:-ON}}
     ^-- SC2223 (info): This default assignment may cause DoS due to globbing. 
Quote it.
   
   
   In ci/scripts/python_test.sh line 48:
   : ${PYARROW_TEST_DATASET:=${ARROW_DATASET:-ON}}
     ^-- SC2223 (info): This default assignment may cause DoS due to globbing. 
Quote it.
   
   
   In ci/scripts/python_test.sh line 49:
   : ${PYARROW_TEST_FLIGHT:=${ARROW_FLIGHT:-ON}}
     ^-- SC2223 (info): This default assignment may cause DoS due to globbing. 
Quote it.
   
   
   In ci/scripts/python_test.sh line 50:
   : ${PYARROW_TEST_GANDIVA:=${ARROW_GANDIVA:-ON}}
     ^-- SC2223 (info): This default assignment may cause DoS due to globbing. 
Quote it.
   
   
   In ci/scripts/python_test.sh line 51:
   : ${PYARROW_TEST_GCS:=${ARROW_GCS:-ON}}
     ^-- SC2223 (info): This default assignment may cause DoS due to globbing. 
Quote it.
   
   
   In ci/scripts/python_test.sh line 52:
   : ${PYARROW_TEST_HDFS:=${ARROW_HDFS:-ON}}
     ^-- SC2223 (info): This default assignment may cause DoS due to globbing. 
Quote it.
   
   
   In ci/scripts/python_test.sh line 53:
   : ${PYARROW_TEST_ORC:=${ARROW_ORC:-ON}}
     ^-- SC2223 (info): This default assignment may cause DoS due to globbing. 
Quote it.
   
   
   In ci/scripts/python_test.sh line 54:
   : ${PYARROW_TEST_PARQUET:=${ARROW_PARQUET:-ON}}
     ^-- SC2223 (info): This default assignment may cause DoS due to globbing. 
Quote it.
   
   
   In ci/scripts/python_test.sh line 55:
   : ${PYARROW_TEST_PARQUET_ENCRYPTION:=${PARQUET_REQUIRE_ENCRYPTION:-ON}}
     ^-- SC2223 (info): This default assignment may cause DoS due to globbing. 
Quote it.
   
   
   In ci/scripts/python_test.sh line 56:
   : ${PYARROW_TEST_S3:=${ARROW_S3:-ON}}
     ^-- SC2223 (info): This default assignment may cause DoS due to globbing. 
Quote it.
   
   
   In ci/scripts/python_test.sh line 72:
   pytest -r s ${PYTEST_ARGS} --pyargs pyarrow
               ^------------^ SC2086 (info): Double quote to prevent globbing 
and word splitting.
   
   Did you mean:
   pytest -r s "${PYTEST_ARGS}" --pyargs pyarrow
   
   
   In ci/scripts/python_test.sh line 76:
     pytest ${PYTEST_RST_ARGS} ${arrow_dir}/docs/source/python
            ^----------------^ SC2086 (info): Double quote to prevent globbing 
and word splitting.
                               ^----------^ SC2086 (info): Double quote to 
prevent globbing and word splitting.
   
   Did you mean:
     pytest "${PYTEST_RST_ARGS}" "${arrow_dir}"/docs/source/python
   
   For more information:
     https://www.shellcheck.net/wiki/SC2034 -- test_dir appears unused. Verify 
u...
     https://www.shellcheck.net/wiki/SC1091 -- Not following: ./bin/activate: 
op...
     https://www.shellcheck.net/wiki/SC2086 -- Double quote to prevent globbing 
...
   ```
   
   
   ### Component(s)
   
   Developer Tools


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to