-
Notifications
You must be signed in to change notification settings - Fork 341
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
refactor(gnolang): remove unnecessary if-else #550
refactor(gnolang): remove unnecessary if-else #550
Conversation
@harry-hov do you plan to make additional ones? if yes, let's keep this PR open and merge regularly by batches. |
i.e., #551 |
@moul I'm not planning to work on an additional PR related to this. At least not this week. I was just going through the |
What about merging 551 into this one and keeping this meta PR open with all the style improvements you find by walking this package and merging it when you finish or on a weekly/bi-weekly basis? |
Sounds good to me. Will close 551. |
…ve-unnecessary-else chore(gnolang/nodes_string): remove unnecessary else
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.
Lookin' good 💯
…ove-unnecessary-else gnolang/values_string: remove unnecessary if-else
…ecessary-else gnolang/values: remove unnecessary if-else
Let's merge this one now, and feel free to open another one we keep open for a few days. Thanks. |
Description
Removed unnecessary else blocks from
pkgs/gnolang
.