Skip to content

Add zizmor to the pre-commit configuration#216

Open
krishkumar6 wants to merge 1 commit into
beeware:mainfrom
krishkumar6:add-zizmor-precommit
Open

Add zizmor to the pre-commit configuration#216
krishkumar6 wants to merge 1 commit into
beeware:mainfrom
krishkumar6:add-zizmor-precommit

Conversation

@krishkumar6

@krishkumar6 krishkumar6 commented Jul 17, 2026

Copy link
Copy Markdown

Adds the zizmor pre-commit hook, and resolves all findings it reports against the GitHub Actions configuration:

  • unpinned-uses: all action and reusable-workflow references are pinned to full commit hashes.
  • excessive-permissions: workflows now default to contents: read, with jobs declaring any elevated permissions they need explicitly.
  • artipacked: checkout no longer persists credentials.
  • secrets-inherit: dependabot-changenote now passes BRUTUS_PAT_TOKEN explicitly instead of inheriting all secrets.
  • dependabot-cooldown: all Dependabot ecosystems use a 7-day cooldown.
  • superfluous-actions: the release is created with "gh release create" instead of ncipollo/release-action.

Fixes #215

PR Checklist:

  • I will abide by the BeeWare Code of Conduct
  • I have read and have followed the CONTRIBUTING.md file
  • This PR was generated or assisted using an AI tool

Assisted-by:

Adds the zizmor pre-commit hook, and resolves all findings it reports
against the GitHub Actions configuration:

* unpinned-uses: all action and reusable-workflow references are pinned
  to full commit hashes.
* excessive-permissions: workflows now default to contents: read, with
  jobs declaring any elevated permissions they need explicitly.
* artipacked: checkout no longer persists credentials.
* secrets-inherit: dependabot-changenote now passes BRUTUS_PAT_TOKEN
  explicitly instead of inheriting all secrets.
* dependabot-cooldown: all Dependabot ecosystems use a 7-day cooldown.
* superfluous-actions: the release is created with "gh release create"
  instead of ncipollo/release-action.

Fixes beeware#215

Co-Authored-By: Claude Fable 5 <[email protected]>

@freakboy3742 freakboy3742 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the PR; however, you'll need to familiarize yourself with our contribution guide. In particular, you haven't used our PR template.

In addition, your change has broken the CI pipeline, so none of the CI tasks are running.

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.

Add zizmor pre-commit configuration

3 participants