Skip to content
This repository was archived by the owner on Jul 16, 2023. It is now read-only.

fix: fix moving to variable false positives #1186

Merged
merged 6 commits into from
Feb 12, 2023

Conversation

incendial
Copy link
Member

Please write a short comment explaining your change (or "none" for internal only changes)

Fixes newly reported false positives for prefer-moving-to-variable.

@incendial incendial added this to the 5.6.0 milestone Feb 11, 2023
@incendial incendial requested a review from dkrutskikh February 11, 2023 13:42
@incendial incendial self-assigned this Feb 11, 2023
@incendial incendial changed the title Fix moving to variable false positives fix: fix moving to variable false positives Feb 11, 2023
@codecov
Copy link

codecov bot commented Feb 11, 2023

Codecov Report

Merging #1186 (2bdc27e) into master (d19a1e4) will increase coverage by 0.04%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #1186      +/-   ##
==========================================
+ Coverage   86.12%   86.16%   +0.04%     
==========================================
  Files         359      359              
  Lines        8085     8112      +27     
==========================================
+ Hits         6963     6990      +27     
  Misses       1122     1122              
Impacted Files Coverage Δ
.../rules_list/prefer_moving_to_variable/visitor.dart 100.00% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@incendial incendial merged commit af1cc2e into master Feb 12, 2023
@incendial incendial deleted the fix-moving-to-variable-false-positives branch February 12, 2023 16:20
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant