Skip to content

ci: add zizmor (pixi task + pre-commit) and fix its auditor findings - #43

Merged
aldbr merged 1 commit into
DIRACGrid:mainfrom
aldbr:ci/zizmor-auditor
Nov 13, 2026
Merged

aldbr merged 1 commit into
DIRACGrid:mainfrom
aldbr:ci/zizmor-auditor

Conversation

@aldbr

@aldbr aldbr commented Nov 12, 2026

Copy link
Copy Markdown
Contributor
  • pixi: gha-lint environment with gha-zizmor and gha-pinact, same as diracx (pixi run -e gha-lint gha-zizmor --no-online-audits --persona=auditor).
  • pre-commit: zizmor hook with the default persona.
  • integration.yml, ci.yml: pass matrix values to shell scripts through env instead of ${{ }} (template-injection; values come from the repo, but env is the recommended pattern).
  • integration.yml: cancel superseded runs of the same ref.
  • name every job and document the release-please write permissions.

zizmor 1.32.0 now reports no findings with both the default and the auditor persona.

- pixi: gha-lint environment with gha-zizmor and gha-pinact, same as diracx
  (`pixi run -e gha-lint gha-zizmor --no-online-audits --persona=auditor`).
- pre-commit: zizmor hook with the default persona.
- integration.yml, ci.yml: pass matrix values to shell scripts through env
  instead of `${{ }}` (template-injection; values come from the repo, but env
  is the recommended pattern).
- integration.yml: cancel superseded runs of the same ref.
- name every job and document the release-please write permissions.

zizmor 1.32.0 now reports no findings with both the default and the auditor
persona.

Assisted-by: Claude:claude-opus-5-5
@aldbr
aldbr merged commit 177aca5 into DIRACGrid:main Nov 13, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants