Skip to content

style: add zizmor config#181

Merged
2bndy5 merged 2 commits intomainfrom
zizmor-config
Jan 22, 2026
Merged

style: add zizmor config#181
2bndy5 merged 2 commits intomainfrom
zizmor-config

Conversation

@2bndy5
Copy link
Contributor

@2bndy5 2bndy5 commented Jan 22, 2026 •
edited by coderabbitai bot
Loading

This should allow pinning cpp-linter reusable workflows to a branch.

See zizmor v1.20 release notes.

Summary by CodeRabbit

  • Chores
    • Updated repository dependency pinning rules for CI/actions to tighten handling of unpinned references across core ecosystems.
    • Expanded spelling configuration: added a project-specific allowed word and extended ignored paths/patterns to reduce false positives during checks.

✏️ Tip: You can customize this high-level summary in your review settings.

@2bndy5
style: add zizmor config
715b9e2 
This should allow pinning cpp-linter reusable workflows to a branch.

See [zizmor v1.20 release notes](https://docs.zizmor.sh/release-notes/#1200).
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 22, 2026 •
edited
Loading

Walkthrough

Added an unpinned-uses policy block to .github/zizmor.yml specifying pin rules for action ecosystems, and updated cspell.config.yml to add the word "zizmor" and expand ignorePaths with additional patterns.

Changes

Cohort / File(s) Summary
Zizmor config
​.github/zizmor.yml		 Added rules.unpinned-uses.config.policies entries: ref-pin for actions/*, github/*, and dependabot/*; allow-any for cpp-linter/*.
Spell checker config
cspell.config.yml		 Added word zizmor to words; expanded ignorePaths with env/**, venv/**, tests/**/*.{json,h,c,cpp,hpp,patch,diff}, **.clang-tidy, **.clang-format, pyproject.toml, .gitignore, and **/*.{yml,yaml,txt}.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'style: add zizmor config' accurately describes the main change: adding a zizmor configuration to the codebase. It is concise, clear, and directly related to the changeset.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@2bndy5 2bndy5 mentioned this pull request Jan 22, 2026
Merged
@2bndy5 2bndy5 merged commit f9b56dc into main Jan 22, 2026
14 checks passed
@2bndy5 2bndy5 deleted the zizmor-config branch January 22, 2026 21:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@2bndy5