Skip to content

Fix CI: clippy 1.97 question_mark + crossbeam-epoch advisory - #21

Merged
fohara merged 1 commit into
mainfrom
fix/ci-clippy-197-and-audit
Jul 14, 2026
Merged

fohara merged 1 commit into
mainfrom
fix/ci-clippy-197-and-audit

Conversation

@fohara

@fohara fohara commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

The main CI run for #20 went red on two jobs, both from toolchain/advisory drift rather than the merged code:

  • Lint (rustc 1.97.0 stable) — clippy::question_mark now fires on two manual Option early-return matches in lash-types/src/path_utils.rs. Collapsed each to the ? operator (behavior-preserving).
  • Security Audit — cargo audit failed on RUSTSEC-2026-0204 (invalid pointer dereference in crossbeam-epoch's fmt::Pointer impl) in the transitive crossbeam-epoch 0.9.18. Bumped to 0.9.20 (patched) via cargo update. The remaining advisories are unmaintained/unsound warnings that the job already allows.

Verified locally: cargo clippy --workspace --all-targets -- -D warnings clean, cargo fmt --all --check clean, cargo audit exits 0, lash-types path_utils tests pass.

🤖 Generated with Claude Code

The Lint job (rustc 1.97 stable) flagged clippy::question_mark on two
manual Option-early-return matches in path_utils; collapse each to the ?
operator. The Security Audit job failed on RUSTSEC-2026-0204 in the
transitive crossbeam-epoch 0.9.18; update it to 0.9.20 (patched).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@fohara
fohara merged commit f6a3990 into main Jul 14, 2026
14 of 15 checks passed
@fohara
fohara deleted the fix/ci-clippy-197-and-audit branch July 14, 2026 14:54
fohara added a commit that referenced this pull request Aug 6, 2026
)

The Lint job (rustc 1.97 stable) flagged clippy::question_mark on two
manual Option-early-return matches in path_utils; collapse each to the ?
operator. The Security Audit job failed on RUSTSEC-2026-0204 in the
transitive crossbeam-epoch 0.9.18; update it to 0.9.20 (patched).
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