-
Notifications
You must be signed in to change notification settings - Fork 88
Lint all Bash scripts to be shellcheck-clean #44
Copy link
Copy link
Open
Labels
AIIssue which can probably be resolved by agentic LLM AI coding, rather than scarce humans.Issue which can probably be resolved by agentic LLM AI coding, rather than scarce humans.BackendOriginal content, Hakyll/Haskell/scripts, Markdown/HTML etc. Usually not CSS/JS. Owner: Gwern.Original content, Hakyll/Haskell/scripts, Markdown/HTML etc. Usually not CSS/JS. Owner: Gwern.enhancementNew feature or requestNew feature or requestgood first issueGood for newcomersGood for newcomers
Metadata
Metadata
Assignees
Labels
AIIssue which can probably be resolved by agentic LLM AI coding, rather than scarce humans.Issue which can probably be resolved by agentic LLM AI coding, rather than scarce humans.BackendOriginal content, Hakyll/Haskell/scripts, Markdown/HTML etc. Usually not CSS/JS. Owner: Gwern.Original content, Hakyll/Haskell/scripts, Markdown/HTML etc. Usually not CSS/JS. Owner: Gwern.enhancementNew feature or requestNew feature or requestgood first issueGood for newcomersGood for newcomers
We enforce
-Wall -Werroron Haskell, and it would be good to have clean safe Bash; all Bash scripts or utilities should be shellcheck-clean.Some of the shellcheck warnings cannot be fixed because they are warning about intended functionality or are just dubious. In those cases, presumably there is some way to whitelist per-line (some sort of comment?), or to add a shellcheck config local to this repo disabling a particular category of warnings.