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

fix: visit only widgets for use-setstate-synchronously #1210

Merged
merged 1 commit into from
Mar 22, 2023

Conversation

incendial
Copy link
Member

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

Fix false-positives for use-setstate-synchronously.

@incendial incendial added this to the 5.7.0 milestone Mar 22, 2023
@incendial incendial self-assigned this Mar 22, 2023
@codecov
Copy link

codecov bot commented Mar 22, 2023

Codecov Report

Merging #1210 (e43c151) into master (a2433c4) will increase coverage by 0.00%.
The diff coverage is 100.00%.

❗ Current head e43c151 differs from pull request most recent head e5615dc. Consider uploading reports for the commit e5615dc to get more accurate results

@@           Coverage Diff           @@
##           master    #1210   +/-   ##
=======================================
  Coverage   86.26%   86.27%           
=======================================
  Files         362      362           
  Lines        8150     8153    +3     
=======================================
+ Hits         7031     7034    +3     
  Misses       1119     1119           
Impacted Files Coverage Δ
...synchronously/use_setstate_synchronously_rule.dart 100.00% <100.00%> (ø)
...rules_list/use_setstate_synchronously/visitor.dart 86.91% <100.00%> (+0.37%) ⬆️

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

@incendial incendial merged commit b9c7b57 into master Mar 22, 2023
@incendial incendial deleted the fix-use-setstate branch March 22, 2023 09:06
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