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

fix: prefer conditional expressions rule #567

Merged
merged 10 commits into from
Nov 25, 2021

Conversation

vlkonoshenko
Copy link
Member

What is the purpose of this pull request? (put an "X" next to an item)

[ ] Documentation update
[X] Bug fix
[ ] New rule
[ ] Changes an existing rule
[ ] Add autofixing to a rule
[ ] Add a CLI option
[ ] Add something to the core
[ ] Other, please explain:

#441

What changes did you make? (Give an overview)

Add checking for increment / decrement operations

Is there anything you'd like reviewers to focus on?

_isIncrementDecrementOperation() function in prefer_conditional_expressions_rule.dart

@vlkonoshenko vlkonoshenko self-assigned this Nov 23, 2021
@codecov
Copy link

codecov bot commented Nov 23, 2021

Codecov Report

Merging #567 (7d37de9) into master (304f56b) will increase coverage by 0.02%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #567      +/-   ##
==========================================
+ Coverage   85.68%   85.71%   +0.02%     
==========================================
  Files         226      226              
  Lines        4682     4689       +7     
==========================================
+ Hits         4012     4019       +7     
  Misses        670      670              
Impacted Files Coverage Δ
...s_list/member_ordering_extended/config_parser.dart 100.00% <100.00%> (ø)
...dering_extended/member_ordering_extended_rule.dart 100.00% <100.00%> (ø)
...s/rules_list/member_ordering_extended/visitor.dart 95.86% <100.00%> (ø)
...pressions/prefer_conditional_expressions_rule.dart 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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

@incendial incendial added area-rules type: enhancement New feature or request labels Nov 24, 2021
@incendial incendial added this to the 4.8.0 milestone Nov 24, 2021
…nal_expressions

# Conflicts:
#	CHANGELOG.md
@dkrutskikh dkrutskikh merged commit 490e032 into master Nov 25, 2021
@dkrutskikh dkrutskikh deleted the 441_prefer_conditional_expressions branch November 25, 2021 19:07
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-rules type: enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants