Add shellcheck to pre-commit and fix warnings

[gforsyth]

Description

shellcheck is a fast, static analysis tool for shell scripts. It's good at
flagging up unused variables, unintentional glob expansions, and other potential
execution and security headaches that arise from the wonders of bash (and
other shlangs).

This PR adds a pre-commit hook to run shellcheck on all of the sh-lang
files in the ci/ directory, and the changes requested by shellcheck to make
the existing files pass the check.

xref: rapidsai/build-planning#135

Checklist

• I am familiar with the Contributing Guidelines.
• New or existing tests cover these changes.
• The documentation is up to date with these changes.

[vyasr]

LGTM, thanks!

[galipremsagar]

@gforsyth I cancelled the most recent workflow to free up resources to unblock all of cudf CI for this PR: #17771

I'll rerun once #17771 is merged.

[gforsyth]

Thanks @galipremsagar -- I'll probably want to rebase on that PR anyway, so that I pick up the polars updates. I'll push that up once CI is a bit less jammed

[gforsyth]

/merge