MartinThoma / flake8-simplify

❄ A flake8 plugin that helps you to simplify code
MIT License
185 stars 19 forks source link

SIM112: Non-capital environment variables #19

Closed MartinThoma closed 3 years ago

MartinThoma commented 3 years ago

Explanation

It is pretty common to have CONSTANT_STYLE environment variables.

Example

# Bad
os.environ["foo"]
os.environ.get("bar")

# Good
os.environ["FOO"]
os.environ.get("BAR")
MartinThoma commented 3 years ago
        Expr(
            value=Subscript(
                value=Attribute(
                    value=Name(id='os', ctx=Load()),
                    attr='environ',
                    ctx=Load(),
                ),
                slice=Index(
                    value=Constant(value='foo', kind=None),
                ),
                ctx=Load(),
            ),
        ),
MartinThoma commented 3 years ago
        Expr(
            value=Call(
                func=Attribute(
                    value=Attribute(
                        value=Name(id='os', ctx=Load()),
                        attr='environ',
                        ctx=Load(),
                    ),
                    attr='get',
                    ctx=Load(),
                ),
                args=[Constant(value='bar', kind=None)],
                keywords=[],
            ),
        ),
MartinThoma commented 3 years ago

Included in v0.10.0