apache / arrow

Apache Arrow is the universal columnar format and multi-language toolbox for fast data interchange and in-memory analytics
https://arrow.apache.org/
Apache License 2.0
14.63k stars 3.56k forks source link

[CI][Dev] Apply ShellCheck lint to all shell scripts #44748

Open kou opened 5 days ago

kou commented 5 days ago

Describe the enhancement requested

This is a meta issue. We should create a sub issue per shell script instead of applying ShellCheck lint to all shell scripts by one large pull request. It's difficult to review.

See #44749 and #44750 as an example.

Sub-issues based on the git ls-files | grep '\.sh$' results:

(We may have more shell scripts.)

Component(s)

Continuous Integration, Developer Tools

pitrou commented 1 day ago

Perhaps we don't need to do it one file at a time to avoid having to create tons of small issues and PRs?

kou commented 11 hours ago

I don't want to review a large PR that includes all changes...