Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

SIM212, SIM220-SIM223: Ternary operator negation #6

Closed
MartinThoma opened this issue Sep 24, 2020 · 0 comments
Closed

SIM212, SIM220-SIM223: Ternary operator negation #6

MartinThoma opened this issue Sep 24, 2020 · 0 comments
Assignees
@MartinThoma
Labels
enhancement New feature or request

Comments

@MartinThoma
Copy link
Owner

Explanation

Removing a negation makes things simpler to understand

Example

# Bad
b if not a else a

# Good
a if a else b
@MartinThoma MartinThoma added the enhancement New feature or request label Sep 24, 2020
@MartinThoma MartinThoma self-assigned this Sep 24, 2020
MartinThoma added a commit that referenced this issue Sep 24, 2020
@MartinThoma
ENH: Add SIM212, SIM220-SIM223
b50ad3f 
Closes #6
@MartinThoma MartinThoma changed the title [New Rule] Ternary operator negation SIM212, SIM220-SIM223: Ternary operator negation Jan 7, 2021
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees

@MartinThoma MartinThoma

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@MartinThoma