Skip to content

chore: remove dead commented util code - #120

Merged
kitsuyui merged 1 commit into
mainfrom
fix/remove-dead-commented-util-code
Jul 5, 2026
Merged

chore: remove dead commented util code#120
kitsuyui merged 1 commit into
mainfrom
fix/remove-dead-commented-util-code

Conversation

@kitsuyui

Copy link
Copy Markdown
Owner

Summary

  • Remove the commented-out deep_clone_* helper block from src/util.rs.
  • Keep the active utility helpers unchanged.

Verification

  • cargo fmt --all -- --check
  • cargo clippy --all-targets --all-features -- -D warnings
  • cargo check
  • cargo test --all-targets --all-features
  • cargo test --all-features
  • cargo build --release --all-features
  • scripts/test-check-cargo-version-tag.sh
  • git diff --check

spellcheck and octocov depend on GitHub Actions reusable workflow/token behavior and will be verified by CI.

@github-actions

Copy link
Copy Markdown

Code Metrics Report

main (0c91c43) #120 (482efa1) +/-
Coverage 68.3% 68.3% 0.0%
Code to Test Ratio 1:0.1 1:0.1 0.0
Test Execution Time 3s 4s +1s
Details
  |                     | main (0c91c43) | #120 (482efa1) | +/-  |
  |---------------------|----------------|----------------|------|
  | Coverage            |          68.3% |          68.3% | 0.0% |
  |   Files             |              4 |              4 |    0 |
  |   Lines             |            550 |            550 |    0 |
  |   Covered           |            376 |            376 |    0 |
  | Code to Test Ratio  |          1:0.1 |          1:0.1 |  0.0 |
  |   Code              |            593 |            593 |    0 |
  |   Test              |             71 |             71 |    0 |
- | Test Execution Time |             3s |             4s |  +1s |

Code coverage of files in pull request scope (58.8% → 58.8%)

Files Coverage +/- Status
src/util.rs 58.8% 0.0% modified

Reported by octocov

@kitsuyui
kitsuyui merged commit a882f04 into main Jul 5, 2026
5 checks passed
@kitsuyui
kitsuyui deleted the fix/remove-dead-commented-util-code branch July 5, 2026 08:55
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