Repository navigation
[pre-commit.ci] pre-commit autoupdate - #137
Conversation
updates: - [github.com/crate-ci/typos: typos-dict-v0.14.2 → v1.51.1](crate-ci/typos@typos-dict-v0.14.2...v1.51.1) - [github.com/rvben/rumdl-pre-commit: v0.2.78 → v0.2.79](rvben/rumdl-pre-commit@v0.2.78...v0.2.79)
Reviewer's guide (collapsed on small PRs)Reviewer's GuideThe PR refreshes two pre-commit dependencies in File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
|
This pull request does not currently match the merge queue conditions, so it cannot be queued from here. The box comes back if it matches again. |
There was a problem hiding this comment.
Hey - I've found 1 issue
Prompt for AI Agents
Please address the comments from this code review:
## Individual Comments
### Comment 1
<location path=".pre-commit-config.yaml" line_range="17" />
<code_context>
- id: md-dead-link-check
- repo: https://github.com/crate-ci/typos
- rev: typos-dict-v0.14.2
+ rev: v1.51.1
hooks:
- id: typos
</code_context>
<issue_to_address>
**New spelling issues pass CI**
When a spelling is flagged by v1.51.1 but not by v1.40.0, the spelling workflow's `crate-ci/typos@v1.40.0` does not report the issue, so CI passes while the v1.51.1 pre-commit hook rejects it; the updated spelling rules are not enforced by CI.
Update the spelling workflow to use Typos v1.51.1.
</issue_to_address>Sourcery assessment
Approval pending. 1 finding to address first.
Blocking findings: .pre-commit-config.yaml:17
| - id: md-dead-link-check | ||
| - repo: https://github.com/crate-ci/typos | ||
| rev: typos-dict-v0.14.2 | ||
| rev: v1.51.1 |
There was a problem hiding this comment.
🟡 Medium · New spelling issues pass CI
When a spelling is flagged by v1.51.1 but not by v1.40.0, the spelling workflow's crate-ci/typos@v1.40.0 does not report the issue, so CI passes while the v1.51.1 pre-commit hook rejects it; the updated spelling rules are not enforced by CI.
Update the spelling workflow to use Typos v1.51.1.
Prompt for AI agents
In `.pre-commit-config.yaml` at line 17:
**New spelling issues pass CI**
When a spelling is flagged by v1.51.1 but not by v1.40.0, the spelling workflow's `crate-ci/typos@v1.40.0` does not report the issue, so CI passes while the v1.51.1 pre-commit hook rejects it; the updated spelling rules are not enforced by CI.
Update the spelling workflow to use Typos v1.51.1.There was a problem hiding this comment.
LlamaPReview — Conditional code-review clear
Reviewed only the available changed regions; no broader coverage is claimed.
Exact-head CI remains unresolved (1 failed, 1 pending); no CI-dependent merge-safety claim is made.
Still coding alone? Open CarbonChat, a chat room right inside Codex.
updates:
Summary by Sourcery
Build: