Skip to content

Fix clippy warnings for Rust 1.62.0 - #139

Merged
morenol merged 1 commit into
smartcorelib:developmentfrom
morenol:lmm/clippy
Jul 14, 2022
Merged

morenol merged 1 commit into
smartcorelib:developmentfrom
morenol:lmm/clippy

Conversation

@morenol

@morenol morenol commented Jul 14, 2022

Copy link
Copy Markdown
Collaborator

No description provided.

@morenol
morenol requested a review from VolodymyrOrlov July 14, 2022 00:51
@codecov-commenter

codecov-commenter commented Jul 14, 2022 •

Copy link
Copy Markdown

Codecov Report

Merging #139 (09d9663) into development (b4a807e) will increase coverage by 0.02%.
The diff coverage is n/a.

@@               Coverage Diff               @@
##           development     #139      +/-   ##
===============================================
+ Coverage        83.40%   83.43%   +0.02%     
===============================================
  Files               78       78              
  Lines             8377     8377              
===============================================
+ Hits              6987     6989       +2     
+ Misses            1390     1388       -2     
Impacted Files Coverage Δ
src/algorithm/sort/heap_select.rs 97.72% <ø> (ø)
src/linear/lasso_optimizer.rs 94.11% <ø> (ø)
src/optimization/first_order/lbfgs.rs 94.44% <0.00%> (+1.58%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b4a807e...09d9663. Read the comment docs.

@morenol

morenol commented Jul 14, 2022

Copy link
Copy Markdown
Collaborator Author

Merging because this is a two lines change PR without any impact for library callers

@morenol
morenol merged commit b482acd into smartcorelib:development Jul 14, 2022
@morenol
morenol deleted the lmm/clippy branch July 14, 2022 01:06
morenol added a commit that referenced this pull request Nov 8, 2022
Co-authored-by: Luis Moreno <morenol@users.noreply.github.com>
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