mirror of
https://github.com/kevinveenbirkenbach/computer-playbook.git
synced 2026-11-06 10:59:59 +00:00
kevinveenbirkenbachandClaude Opus 5
6fee7ca136
Four parse checks join the `lint-*` family, each using the language's own parser rather than a reimplementation: `php -l`, `ruby -c`, `sqlfluff parse`, and stylelint with an empty rule set (it raises CssSyntaxError whatever the rules say, so the check stays syntax-only and grows no style opinion). Three unit runners join `test-unit`: node:test, PHPUnit, minitest -- the first and third ship with their runtime, so only PHPUnit adds a dependency. Provisioning is best effort and the lint script decides whether it can run. A machine without PHP must not fail the gate, but it must not look like success either: the skip names how many files went unchecked. The first version of these scripts read `git ls-files`, which inside the compose container returns nothing (git refuses the bind-mounted repo as "dubious ownership") and reported "No SQL files found." over 52 files -- green, and a lie. They use `find` now, as the surrounding lint scripts already did. sqlfluff needed two corrections before it parsed anything real. `%(name)s` is a psycopg parameter marker, not SQL, so the placeholder templater is configured rather than the files exempted; and the dialect follows each role's declared engine (`meta/services.yml`), which is postgres for six roles and mysql for svc-db-mariadb and pixelfed. Recipes stay in scripts, not in the Makefile: the targets delegate, matching every other target in the file. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
25 lines
715 B
Python
25 lines
715 B
Python
"""Install sqlfluff via pip (parser for lint-sql).
|
|
|
|
Provisioning is best effort: a host that cannot reach PyPI must not fail the
|
|
whole gate. scripts/lint/sql.sh skips loudly instead.
|
|
"""
|
|
|
|
from __future__ import annotations
|
|
|
|
from utils.install.pip import install_pip_pkg
|
|
from utils.install.primitives import log, warn, which
|
|
|
|
|
|
def ensure() -> None:
|
|
if which("sqlfluff"):
|
|
return
|
|
|
|
log("Missing command 'sqlfluff'. Installing via pip.")
|
|
try:
|
|
install_pip_pkg("sqlfluff")
|
|
except (RuntimeError, OSError) as exc:
|
|
warn(f"[install-lint] sqlfluff could not be installed: {exc}")
|
|
|
|
if not which("sqlfluff"):
|
|
warn("[install-lint] sqlfluff is unavailable; lint-sql will skip.")
|