tactcomplabs / rev

RISC-V SST CPU Component
Other
17 stars 21 forks source link

Cleanup shellcheck errors #328

Closed leekillough closed 3 weeks ago

leekillough commented 3 weeks ago

This fixes warnings from the shellcheck linter.

All default and optional warnings except a few have been enabled.

In all of the CI Slurm scripts, multiple redirections in multiple commands were replaced with a single exec redirection, which redirects the current shell and its children.

.githooks/pre-commit warns if you don't have shellcheck installed -- it can be installed with brew install shellcheck or apt-get install shellcheck.