Skip to content

[CI][Dev] Fix shellcheck errors in the ci/scripts/csharp_test.sh #46794

Closed
@hiroyuki-sato

Description

@hiroyuki-sato

Describe the enhancement requested

This is the sub issue #44748.

  • SC2155: Declare and assign separately to avoid masking return values.
  • SC2086: Double quote to prevent globbing and word splitting.
shellcheck ci/scripts/csharp_test.sh

In ci/scripts/csharp_test.sh line 33:
export PYTHONNET_PYDLL=$(${PYTHON} -m find_libpython)
       ^-------------^ SC2155 (warning): Declare and assign separately to avoid masking return values.


In ci/scripts/csharp_test.sh line 35:
pushd ${source_dir}
      ^-----------^ SC2086 (info): Double quote to prevent globbing and word splitting.

Did you mean:
pushd "${source_dir}"

For more information:
  https://www.shellcheck.net/wiki/SC2155 -- Declare and assign separately to ...
  https://www.shellcheck.net/wiki/SC2086 -- Double quote to prevent globbing ...

Component(s)

Continuous Integration

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

Milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions