Skip to content

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

Merged
wyli merged 2 commits intodeepsource-fix-1eb41095from
deepsource-fix-abb1815a
Aug 7, 2021
Merged

Refactor unnecessary else / elif when if block has a return statement#2717
wyli merged 2 commits intodeepsource-fix-1eb41095from
deepsource-fix-abb1815a

Conversation

@deepsource-autofix
Copy link
Copy Markdown
Contributor

No description provided.

@wyli wyli changed the base branch from dev to deepsource-fix-1eb41095 August 7, 2021 13:49
@wyli wyli merged commit 5165d09 into deepsource-fix-1eb41095 Aug 7, 2021
@wyli wyli deleted the deepsource-fix-abb1815a branch August 7, 2021 13:49
wyli pushed a commit that referenced this pull request Aug 7, 2021
* Remove unnecessary use of comprehension

* Remove unnecessary comprehension (#2718)

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

* Refactor unnecessary `else` / `elif` when `if` block has a `return` statement (#2717)

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

* Unnecessary `else`/`elif` used after `raise` PYL-R1720

Signed-off-by: Wenqi Li <[email protected]>

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

* Replace ternary syntax with if expression (#2716)

* Replace ternary syntax with if expression

* fixes Duplicate dictionary keys PYL-W0109

Signed-off-by: Wenqi Li <[email protected]>

* remove Unused variable data_

Signed-off-by: Wenqi Li <[email protected]>

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

* [MONAI] python code formatting

Signed-off-by: monai-bot <[email protected]>

* fixes scaler type

Signed-off-by: Wenqi Li <[email protected]>

Co-authored-by: deepsource-autofix[bot] <62050782+deepsource-autofix[bot]@users.noreply.github.com>
Co-authored-by: monai-bot <[email protected]>
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