-
-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Conditional assignment #2416
Conditional assignment #2416
Conversation
13cf4e8
to
0154441
Compare
Description: >- | ||
Use the return value of `if` and `case` statements for | ||
assignment to a variable instead of assigning that variable | ||
inside of each branch |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Small tweak: end sentence with a period.
👍 |
0154441
to
15a2d3d
Compare
15a2d3d
to
b8bb1f0
Compare
31st of July? :-) This has been in the works for quite a while. :-) |
Yeah, I have been working on this on and off for quite a while. The design has completely changed on it at least twice. |
The code is a bit dense (a few comments here and there might be useful), but overall it looks OK to me. I noticed the tests were pretty exhaustive and that's the most important thing. 👍 |
I will add some comments and submit a new PR. |
No description provided.