Skip to content

fix: update stale license headers and resolve clippy pedantic warnings#172

Merged
pdf-amzn merged 1 commit into
mainfrom
fix/license-headers-and-clippy
Jun 11, 2026
Merged

fix: update stale license headers and resolve clippy pedantic warnings#172
pdf-amzn merged 1 commit into
mainfrom
fix/license-headers-and-clippy

Conversation

@amrith

@amrith amrith commented Jun 8, 2026

Copy link
Copy Markdown
Collaborator
  • Update 7 files with old 'DynamoDB Open' copyright to 'ExtendDB contributors'
  • Rename _backend field to backend (resolves underscore-prefix warnings)
  • Apply clippy auto-fixes: redundant closures, format string variables, unnecessary raw string hashes, missing backtick docs, must_use attributes
  • Rewrite match-to-let-else patterns (6 instances)
  • Fix long literal lacking separators in build.rs
  • Fix Map::default() clarity warning

Clippy pedantic count: 504 -> 171 (66% reduction)

What

Why

Closes #

Testing done

Checklist

  • I have read CONTRIBUTING.md
  • All tests pass (cargo test --workspace)
  • Code is formatted (cargo fmt --check)
  • Clippy is clean (cargo clippy -- -W clippy::pedantic)
  • I have added or updated tests for new functionality
  • I have updated documentation if behavior changed
  • Breaking changes are noted below (if any)
  • If this changes the wire protocol, Storage trait, auth model, on-disk
    format, or public CLI surface, an RFC has been accepted or is linked
    below. Otherwise, an ADR captures the decision (link below).

ADR / RFC:

Breaking changes


By submitting this pull request, I confirm that my contribution is made under
the terms of the Apache License 2.0 and I agree to the Developer Certificate of
Origin (DCO). See CONTRIBUTING.md for details.

@amrith amrith force-pushed the fix/license-headers-and-clippy branch 3 times, most recently from 4e2ec16 to 76b4ad9 Compare June 11, 2026 04:24
- Update 7 files with old 'DynamoDB Open' copyright to 'ExtendDB contributors'
- Rename _backend field to backend (resolves underscore-prefix warnings)
- Apply clippy auto-fixes: redundant closures, format string variables,
  unnecessary raw string hashes, missing backtick docs, must_use attributes
- Rewrite match-to-let-else patterns (6 instances)
- Fix long literal lacking separators in build.rs
- Fix Map::default() clarity warning

Clippy pedantic count: 504 -> 171 (66% reduction)
@pdf-amzn pdf-amzn force-pushed the fix/license-headers-and-clippy branch from 76b4ad9 to b52d87d Compare June 11, 2026 17:50
@pdf-amzn pdf-amzn added this pull request to the merge queue Jun 11, 2026
Merged via the queue into main with commit 28cdd83 Jun 11, 2026
11 checks passed
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