Skip to content

Add yamllint, codespell, zizmor, and actionlint pre-commit hooks - #501

Merged
hluk merged 1 commit into
release-engineering:masterfrom
hluk:normalize-pre-commit-hooks
Aug 4, 2026
Merged

Add yamllint, codespell, zizmor, and actionlint pre-commit hooks#501
hluk merged 1 commit into
release-engineering:masterfrom
hluk:normalize-pre-commit-hooks

Conversation

@hluk

@hluk hluk commented Aug 4, 2026

Copy link
Copy Markdown
Member

Add missing linter hooks and fix issues they found:

  • yamllint with lenient indentation config
  • codespell for spell checking
  • zizmor and actionlint for GitHub Actions workflows
  • Fix typos found by codespell
  • Add persist-credentials: false to checkout actions
  • Add permissions block to GitHub Actions workflow

Assisted-by: Claude (Anthropic)

Add missing linter hooks and fix issues they found:
- yamllint with lenient indentation config
- codespell for spell checking
- zizmor and actionlint for GitHub Actions workflows
- Fix typos found by codespell
- Add persist-credentials: false to checkout actions
- Add permissions block to GitHub Actions workflow

Assisted-by: Claude (Anthropic)
@codecov

codecov Bot commented Aug 4, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 92.54%. Comparing base (d34c821) to head (48d3ddf).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #501   +/-   ##
=======================================
  Coverage   92.54%   92.54%           
=======================================
  Files          55       55           
  Lines        4279     4279           
  Branches      270      270           
=======================================
  Hits         3960     3960           
  Misses        264      264           
  Partials       55       55           
Flag Coverage Δ
unit-tests 92.54% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@hluk
hluk merged commit 88328f7 into release-engineering:master Aug 4, 2026
11 checks passed
@hluk hluk changed the title Add yamllint, codespell, zizmor, and actionlint pre-commit hooks Remove bandit tox env and unnecessary bandit args Aug 4, 2026
@hluk hluk changed the title Remove bandit tox env and unnecessary bandit args Add yamllint, codespell, zizmor, and actionlint pre-commit hooks Aug 4, 2026
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.

1 participant