Skip to content

fix: address clippy 1.95.0 lint errors#728

Merged
ptondereau merged 2 commits intoextphprs:masterfrom
ptondereau:fix/clippy-1-95-0
Apr 20, 2026
Merged

fix: address clippy 1.95.0 lint errors#728
ptondereau merged 2 commits intoextphprs:masterfrom
ptondereau:fix/clippy-1-95-0

Conversation

@ptondereau
Copy link
Copy Markdown
Member

@ptondereau ptondereau commented Apr 20, 2026

Description

Fixes clippy 1.95.0 lint errors

Checklist

- drop trailing comma in `panic!` (unnecessary_trailing_comma)
- remove redundant `as u32` casts on already-`u32` expressions
  in `exception_observer` (unnecessary_cast)
- collapse nested `if` into match arm guard in cargo-php installer
  (collapsible_match)
Aligns the devshell toolchain with CI stable (1.95.0) so local
`cargo clippy` surfaces the same lints the workflow enforces.

- nixpkgs: 46db2e09 -> 4bd9165a
- rust-overlay: 75900435 -> 25d75be8
@coveralls
Copy link
Copy Markdown

coveralls commented Apr 20, 2026

Coverage Report for CI Build 24656691002

Coverage increased (+0.005%) to 65.9%

Details

  • Coverage increased (+0.005%) from the base build.
  • Patch coverage: 5 uncovered changes across 3 files (0 of 5 lines covered, 0.0%).
  • No coverage regressions found.

Uncovered Changes

File Changed Covered %
crates/cli/src/lib.rs 2 0 0.0%
src/zend/exception_observer.rs 2 0 0.0%
src/builders/enum_builder.rs 1 0 0.0%

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 12994
Covered Lines: 8563
Line Coverage: 65.9%
Coverage Strength: 32.96 hits per line

💛 - Coveralls

@github-actions
Copy link
Copy Markdown

🐰 Bencher Report

Branchfix/clippy-1-95-0
TestbedPHP 8.4.20 (cli) (built: Apr 11 2026 00:52:25) (NTS)

⚠️ WARNING: Truncated view!

The full continuous benchmarking report exceeds the maximum length allowed on this platform.

⚠️ WARNING: No Threshold found!

Without a Threshold, no Alerts will ever be generated.

🐰 View full continuous benchmarking report in Bencher

@ptondereau ptondereau merged commit 25e4274 into extphprs:master Apr 20, 2026
66 checks passed
@ptondereau ptondereau deleted the fix/clippy-1-95-0 branch April 20, 2026 08:45
@Xenira Xenira mentioned this pull request Apr 15, 2026
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.

2 participants