Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

chore: fix some clippy lints #184

Merged
merged 1 commit into from
Jul 8, 2024
Merged

chore: fix some clippy lints #184

merged 1 commit into from
Jul 8, 2024

Conversation

tedil
Copy link
Contributor

@tedil tedil commented Jul 4, 2024

No description provided.

@tedil tedil requested a review from holtgrewe July 4, 2024 12:28
Copy link

codecov bot commented Jul 4, 2024

Codecov Report

Attention: Patch coverage is 70.00000% with 3 lines in your changes missing coverage. Please review.

Project coverage is 86.31%. Comparing base (d59783c) to head (4becad7).
Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #184      +/-   ##
==========================================
- Coverage   86.32%   86.31%   -0.02%     
==========================================
  Files          18       18              
  Lines        7137     7139       +2     
==========================================
+ Hits         6161     6162       +1     
- Misses        976      977       +1     
Files Coverage Δ
src/data/cdot/json.rs 89.59% <ø> (ø)
src/normalizer.rs 70.73% <100.00%> (ø)
src/parser/ds.rs 64.97% <50.00%> (-0.13%) ⬇️

@tedil tedil merged commit 1e9b0aa into main Jul 8, 2024
7 of 8 checks passed
@tedil tedil deleted the clippy-lints branch July 8, 2024 14:43
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant