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: minor refactors #1619

Merged
merged 1 commit into from
Nov 26, 2023
Merged

Conversation

TimothyMakkison
Copy link
Contributor

Splitting up #1596, tried to avoid modifying the whitespace.

  • use TryGetValue over ContainsKey/lookup
  • Remove unneeded array duplication via ToArray
  • Invert ifs to improve readability

Copy link

codecov bot commented Nov 21, 2023

Codecov Report

Attention: 11 lines in your changes are missing coverage. Please review.

Comparison is base (2dff048) 87.85% compared to head (10f0134) 87.81%.

Files Patch % Lines
Refit/RequestBuilderImplementation.cs 81.13% 7 Missing and 3 partials ⚠️
Refit/FormValueMultimap.cs 97.43% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1619      +/-   ##
==========================================
- Coverage   87.85%   87.81%   -0.04%     
==========================================
  Files          33       33              
  Lines        2354     2347       -7     
  Branches      295      294       -1     
==========================================
- Hits         2068     2061       -7     
  Misses        207      207              
  Partials       79       79              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@glennawatson
Copy link
Contributor

Happy to merge this one in when we get it compiling. Thanks

@TimothyMakkison
Copy link
Contributor Author

TimothyMakkison commented Nov 26, 2023

Sorry, ran csharpier one more time. No potential logic changes 👍

@glennawatson glennawatson merged commit 3cbe67a into reactiveui:main Nov 26, 2023
2 of 3 checks passed
@TimothyMakkison TimothyMakkison deleted the refactor branch November 26, 2023 23:30
Copy link

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 11, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants