Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Chore/fix clippy perf warnings #4567

Merged
merged 1 commit into from Mar 20, 2024

Conversation

ASuciuX
Copy link
Contributor

@ASuciuX ASuciuX commented Mar 20, 2024

Description

Replaces expect_fun_call with unwrap_or_else.

Applicable issues

@ASuciuX ASuciuX changed the base branch from master to next March 20, 2024 14:39
@ASuciuX ASuciuX requested a review from jferrant March 20, 2024 14:41
@ASuciuX ASuciuX self-assigned this Mar 20, 2024
@ASuciuX ASuciuX requested a review from jbencin March 20, 2024 14:41
Copy link

codecov bot commented Mar 20, 2024

Codecov Report

Attention: Patch coverage is 83.33333% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 71.72%. Comparing base (80adc27) to head (497fec3).

Additional details and impacted files
@@             Coverage Diff             @@
##             next    #4567       +/-   ##
===========================================
- Coverage   83.35%   71.72%   -11.64%     
===========================================
  Files         453      453               
  Lines      328491   328491               
  Branches      323      323               
===========================================
- Hits       273822   235616    -38206     
- Misses      54661    92867    +38206     
  Partials        8        8               
Files Coverage Δ
stacks-signer/src/signer.rs 74.79% <83.33%> (-0.48%) ⬇️

... and 233 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

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

@ASuciuX ASuciuX added this pull request to the merge queue Mar 20, 2024
Merged via the queue into stacks-network:next with commit 468cb30 Mar 20, 2024
1 of 2 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.

Fix Clippy perf for stacks-signer
3 participants