Skip to content

chore: Add clippy lints and fix violations - #6

Merged
joshuaclayton merged 1 commit into
mainfrom
fix-categories
Mar 31, 2026
Merged

chore: Add clippy lints and fix violations#6
joshuaclayton merged 1 commit into
mainfrom
fix-categories

Conversation

@joshuaclayton

Copy link
Copy Markdown
Owner

New lints:

  • unwrap_used: complements existing expect_used; prevents .unwrap() in
    library code
  • wildcard_enum_match_arm: complements exhaustive_enums; forces explicit
    variant handling instead of _ catch-alls
  • missing_errors_doc: enforces # Errors sections on public Result-returning fns
  • missing_panics_doc: enforces # Panics sections where applicable
  • redundant_else: removes else after blocks that already return/break

Fixes:

  • Add unwrap_used to test module expect lists (alongside existing expect_used)
  • Expand wildcard match arms in jwks.rs to list variants explicitly
  • Replace wildcard match in pages.rs error formatting with if-let
  • Replace match/panic patterns in page tests with assert!(matches!(...))

Co-Authored-By: Claude Opus 4.6 (1M context) noreply@anthropic.com

New lints:
- unwrap_used: complements existing expect_used; prevents .unwrap() in
  library code
- wildcard_enum_match_arm: complements exhaustive_enums; forces explicit
  variant handling instead of _ catch-alls
- missing_errors_doc: enforces # Errors sections on public Result-returning fns
- missing_panics_doc: enforces # Panics sections where applicable
- redundant_else: removes else after blocks that already return/break

Fixes:
- Add unwrap_used to test module expect lists (alongside existing expect_used)
- Expand wildcard match arms in jwks.rs to list variants explicitly
- Replace wildcard match in pages.rs error formatting with if-let
- Replace match/panic patterns in page tests with assert!(matches!(...))

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@joshuaclayton
joshuaclayton merged commit 90a6537 into main Mar 31, 2026
8 checks passed
@joshuaclayton
joshuaclayton deleted the fix-categories branch March 31, 2026 15:05
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