-
Notifications
You must be signed in to change notification settings - Fork 39
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
Bug in fix mode in rule AVOID_NULL_CHECK
#1293
Labels
bug
Something isn't working
Comments
minimal reproducible example, I believe the whole body of Also, please provide original code when reporting autofixer issues.
Please specify more explicitly, whether fixed code has diktat errors or syntax errors, like here |
In save-cloud we found out that construction
is not fixed into
Need to investigate. |
3 tasks
sanyavertolet
added a commit
that referenced
this issue
Oct 26, 2022
### What's done: * Added isAssignment utility function to check if a line has assignment operator ('=') using simple regex * Added a check for assignment operator when deciding if to put run block or not * Added tests (#1293)
3 tasks
sanyavertolet
added a commit
that referenced
this issue
Oct 26, 2022
### What's done: * Added isAssignment utility function to check if a line has assignment operator ('=') using simple regex * Added a check for assignment operator when deciding if to put run block or not * Added tests (#1293)
sanyavertolet
added a commit
that referenced
this issue
Oct 26, 2022
### What's done: * Added isAssignment utility function to check if a line has assignment operator ('=') using simple regex * Added a check for assignment operator when deciding if to put run block or not * Added tests (#1293)
sanyavertolet
added a commit
that referenced
this issue
Oct 27, 2022
### What's done: * Added a check for assignment operator when deciding if to put run block or not * Added tests (#1293)
sanyavertolet
added a commit
that referenced
this issue
Oct 27, 2022
### What's done: * Added a check for assignment operator when deciding if to put run block or not * Added tests (#1293)
sanyavertolet
added a commit
that referenced
this issue
Oct 27, 2022
* Bugfix for fix mode of AVOID_NULL_CHECK ### What's done: * Added a check for assignment operator when deciding if to put run block or not * Added tests (#1293)
nulls
pushed a commit
that referenced
this issue
Oct 27, 2022
* Bugfix for fix mode of AVOID_NULL_CHECK * Added a check for assignment operator when deciding if to put run block or not * Added tests (#1293)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Describe the bug
Rule
AVOID_NULL_CHECK
in fix mode give resultBut then an error is Variable Expected
Expected behavior
Code should be like this
Environment information
The text was updated successfully, but these errors were encountered: