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

Cleanup unnecessary syntax #1854

Merged
merged 4 commits into from
Feb 22, 2021
Merged

Cleanup unnecessary syntax #1854

merged 4 commits into from
Feb 22, 2021

Conversation

wcekan
Copy link
Contributor

@wcekan wcekan commented Feb 21, 2021

Description

Reduce code clutter by cleaning up syntax.
A lambda with a single statement does not need a code block.
Removed unnecessary else following a return to improve readability.

Motivation and Context

Code quality.

How Has This Been Tested?

Unit testing.

License

I confirm that this contribution is made under an Apache 2.0 license and that I have the authority necessary to make this contribution on behalf of its copyright owner.

Copy link
Collaborator

@moizarafat moizarafat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@aklish aklish merged commit e2005d6 into master Feb 22, 2021
@aklish aklish deleted the cleanup-syntax branch February 22, 2021 16:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants