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

Refactor unnecessary else / elif when if block has a return statement #2996

Merged
merged 5 commits into from
Sep 22, 2021

Conversation

deepsource-autofix[bot]
Copy link
Contributor

No description provided.

deepsource-autofix bot and others added 5 commits September 21, 2021 21:01
Co-authored-by: deepsource-autofix[bot] <62050782+deepsource-autofix[bot]@users.noreply.github.com>
Signed-off-by: Wenqi Li <wenqil@nvidia.com>
Signed-off-by: Wenqi Li <wenqil@nvidia.com>
Signed-off-by: Wenqi Li <wenqil@nvidia.com>
@wyli wyli force-pushed the deepsource-fix-56559bf7 branch from 3d1dcc4 to 6f52383 Compare September 21, 2021 21:32
Copy link
Contributor

@wyli wyli left a comment

Choose a reason for hiding this comment

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

nonbreaking fixes suggested by the automated tools

@wyli wyli enabled auto-merge (squash) September 22, 2021 01:10
@wyli wyli merged commit 790fc8f into dev Sep 22, 2021
@wyli wyli deleted the deepsource-fix-56559bf7 branch September 22, 2021 01:52
wyli added a commit that referenced this pull request Sep 22, 2021
…tatement (#2996)

* Refactor unnecessary `else` / `elif` when `if` block has a `return` statement

* Remove unnecessary lambda expression (#2997)

Co-authored-by: deepsource-autofix[bot] <62050782+deepsource-autofix[bot]@users.noreply.github.com>

* type fixes

Signed-off-by: Wenqi Li <wenqil@nvidia.com>

* remove repeated init

Signed-off-by: Wenqi Li <wenqil@nvidia.com>

* fixes import, unused vars

Signed-off-by: Wenqi Li <wenqil@nvidia.com>

Co-authored-by: deepsource-autofix[bot] <62050782+deepsource-autofix[bot]@users.noreply.github.com>
Co-authored-by: Wenqi Li <wenqil@nvidia.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant