epic: shellcheck passes on every shell script, with no inline disables #766

New issue

Open

opened 2026-10-14 19:15:24 +00:00 by hexajon · 0 comments

hexajon commented

2026-10-14 19:15:24 +00:00

Owner

The repo carries 27 inline # shellcheck disable directives, and the gate lints only 1 of the 52 tracked .sh files. CONTRIBUTING.md:319 says that linters flag real issues and that I fix the code.

The children are linked as dependencies of this issue. The gate step comes first, so that the gate checks each removal.

Done when every child is closed, the gate's shellcheck step passes on every tracked .sh file, and no inline shellcheck disable remains in the repo.

The repo carries 27 inline `# shellcheck disable` directives, and the gate lints only 1 of the 52 tracked `.sh` files. `CONTRIBUTING.md:319` says that linters flag real issues and that I fix the code. The children are linked as dependencies of this issue. The gate step comes first, so that the gate checks each removal. Done when every child is closed, the gate's shellcheck step passes on every tracked `.sh` file, and no inline shellcheck disable remains in the repo.

hexajon added a new dependency

2026-10-14 19:15:29 +00:00

#761 gate: lint every shell script with shellcheck, and fail when shellcheck is missing

hexajon added a new dependency

2026-10-14 19:15:29 +00:00

#762 lint: remove the SC1091 disables from the shell scripts

hexajon added a new dependency

2026-10-14 19:15:30 +00:00

#763 lint: remove the SC2317 disables from gate.sh and mcp-conformance.sh

hexajon added a new dependency

2026-10-14 19:15:30 +00:00

#764 lint: remove the SC2016 disables from the hook tests