Skip to content

Reduce level of nesting #128739

@byBenPuls

Description

@byBenPuls

Feature or enhancement

Proposal:

Make code easier to read by reducing the level of nesting. High levels of nesting make code more confusing and less readable.
For example:

if process.returncode == 0:
    return stdout.decode(*DECODE_ARGS)
else:
    raise CalledProcessError(
        process.returncode, args,
        stdout.decode(*DECODE_ARGS), stderr.decode(*DECODE_ARGS)
    )

We can remove else, simplify the code:

if process.returncode == 0:
    return stdout.decode(*DECODE_ARGS)
raise CalledProcessError(
    process.returncode, args,
    stdout.decode(*DECODE_ARGS), stderr.decode(*DECODE_ARGS)
)

Has this already been discussed elsewhere?

This is a minor feature, which does not need previous discussion elsewhere

Links to previous discussion of this feature:

No response

Metadata

Metadata

Assignees

No one assigned

    Labels

    type-featureA feature request or enhancement

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions