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

Fix simplification of boolean expression #1523

Merged
merged 8 commits into from
Sep 15, 2022

Conversation

nulls
Copy link
Member

@nulls nulls commented Sep 14, 2022

What's done:

  • added test

It closes #1521

@nulls nulls self-assigned this Sep 14, 2022
@github-actions
Copy link
Contributor

github-actions bot commented Sep 14, 2022

JUnit Tests (macOS, EnricoMi/publish-unit-test-result-action@v1)

1 923 tests   1 908 ✔️  3m 47s ⏱️
   159 suites       15 💤
   159 files           0

Results for commit 1817be4.

♻️ This comment has been updated with latest results.

@github-actions
Copy link
Contributor

github-actions bot commented Sep 14, 2022

JUnit Tests (Windows, EnricoMi/publish-unit-test-result-action@v1)

1 938 tests   1 923 ✔️  2m 54s ⏱️
   160 suites       15 💤
   160 files           0

Results for commit 1817be4.

♻️ This comment has been updated with latest results.

@nulls nulls force-pushed the bugfix/expression_simplification#1521 branch from a16a21b to be9a203 Compare September 14, 2022 15:38
@codecov
Copy link

codecov bot commented Sep 15, 2022

Codecov Report

Merging #1523 (1817be4) into master (52b4096) will increase coverage by 0.03%.
The diff coverage is 96.29%.

@@             Coverage Diff              @@
##             master    #1523      +/-   ##
============================================
+ Coverage     82.55%   82.59%   +0.03%     
  Complexity     2577     2577              
============================================
  Files           111      111              
  Lines          7735     7756      +21     
  Branches       2125     2127       +2     
============================================
+ Hits           6386     6406      +20     
  Misses          443      443              
- Partials        906      907       +1     
Flag Coverage Δ
unittests 82.59% <96.29%> (+0.03%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...t/ruleset/rules/chapter3/BooleanExpressionsRule.kt 92.56% <94.11%> (-0.10%) ⬇️
...tlin/org/cqfn/diktat/ruleset/utils/AstConstants.kt 100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@nulls nulls requested review from petertrr and orchestr7 September 15, 2022 09:31
@nulls nulls added bug Something isn't working autofix Issues related to diktat in autofix mode labels Sep 15, 2022
@nulls nulls marked this pull request as ready for review September 15, 2022 09:32
@nulls nulls enabled auto-merge (squash) September 15, 2022 09:58
@nulls nulls merged commit cf9bce0 into master Sep 15, 2022
@nulls nulls deleted the bugfix/expression_simplification#1521 branch September 15, 2022 12:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
autofix Issues related to diktat in autofix mode bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Small issue on boolean expressions simplification rule (De Morgan's law)
2 participants