Repository navigation
unpinned-tools, docs: fix typos - #2008
Conversation
There was a problem hiding this comment.
Pull request overview
Simple typo fix changing "implictly" to "implicitly" in the unpinned-tools audit finding message, with corresponding updates to test snapshot baselines.
Changes:
- Fix typo in finding annotation message in
unpinned_tools.rs - Update integration test baselines to match corrected spelling
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| crates/zizmor/src/audit/unpinned_tools.rs | Corrects "implictly" → "implicitly" in finding message |
| crates/zizmor/tests/integration/audit/unpinned_tools.rs | Updates test baselines to reflect corrected spelling |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
martincostello
changed the title
Fix typo in finding message.
Update `installaton` to `installation`.
|
Thank you @martincostello! |
|
P.S. in the future please don't request a review from Copilot, it sends me a lot of email notifications 🙂 (only one from each PR, but a lot in aggregate) |
I didn't explicitly ask Copilot to review for this specific PR, but I do have this Copilot setting enabled at my account level: 
|
|
Ah, I'm kind of surprised that gets honored across third-party repos. I guess I'll see if there's a way I can disable it on the organization level here. |
Pre-submission checks
Please check these boxes:
Mandatory: This PR corresponds to an issue (if not, please create
one first).
Having read the AI policy, I hereby disclose the use of an LLM or other
AI coding assistant in the creation of this PR. PRs will not be rejected
for using AI tools, but will be rejected for undisclosed use or
use that violates the policy.
If a checkbox is not applicable, you can leave it unchecked.
Summary
Fix typos found while reporting #2007 and #2009.
Test Plan
Visual inspection and updated the typo in the baselines.