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

fix: handle dynamics in map literals for avoid-dynamic #1194

Merged
merged 4 commits into from
Mar 20, 2023
Merged

Conversation

incendial
Copy link
Member

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

#1191

@incendial incendial added this to the 5.7.0 milestone Feb 18, 2023
@incendial incendial requested a review from dkrutskikh February 18, 2023 11:08
@incendial incendial self-assigned this Feb 18, 2023
@codecov
Copy link

codecov bot commented Feb 18, 2023

Codecov Report

Merging #1194 (6cd6b17) into master (85be501) will increase coverage by 0.00%.
The diff coverage is 100.00%.

❗ Current head 6cd6b17 differs from pull request most recent head 7d3f0ad. Consider uploading reports for the commit 7d3f0ad to get more accurate results

@@           Coverage Diff           @@
##           master    #1194   +/-   ##
=======================================
  Coverage   86.29%   86.29%           
=======================================
  Files         363      363           
  Lines        8179     8182    +3     
=======================================
+ Hits         7058     7061    +3     
  Misses       1121     1121           
Impacted Files Coverage Δ
...les_list/avoid_ignoring_return_values/visitor.dart 96.96% <ø> (ø)
...k_for_equals_in_render_object_setters/visitor.dart 97.43% <ø> (ø)
.../rules_list/prefer_moving_to_variable/visitor.dart 100.00% <ø> (ø)
.../anti_patterns/anti_patterns_list/long_method.dart 100.00% <100.00%> (ø)
...tterns/anti_patterns_list/long_parameter_list.dart 100.00% <100.00%> (ø)
...alyzer/rules/rules_list/avoid_dynamic/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 1edfd17 into master Mar 20, 2023
@incendial incendial deleted the fix-dynamics branch March 20, 2023 15:17
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