Open
Description
Since #10787, we are now running shellcheck on scripts.
Just like with C/C++ formatting, we are excluding almost all existing files by default. However, the reason for doing that is different than for formatting --- fixing shellcheck lint errors does not have the downside of causing merge conflicts churn, but does require manual edits. Thus we're skipping existing files not because it would be potentially bad to get them passing, but because it is more work than is appropriate for a single PR.
Scripts can one-by-one be made passing, which is a great issue to share between new contributors!