Skip to main content

Check for python builtins being used as variables or parameters.

Project description

https://travis-ci.org/gforcada/flake8-builtins.svg?branch=master https://coveralls.io/repos/gforcada/flake8-builtins/badge.svg?branch=master&service=github

Flake8 Builtins plugin

Check for python builtins being used as variables or parameters.

Imagine some code like this:

def max_values(list, list2):
    max = list[0]
    for x in list:
        if x > 0:
            max = x

    all_values = list()
    all_values.append(max)

    max = list2[0]
    for x in list2:
        if x > 0:
            max = x
    all_values.append(max)

    return all_values

max_values([3, 4, 5, ], [5, 6, 7])

The last statement is not returning [5, 7] as one would expect, instead is raising this exception:

Traceback (most recent call last):
  File "test.py", line 17, in <module>
    max_values([3,4,5], [4,5,6])
  File "bla.py", line 6, in max_values
    all_values = list()
TypeError: 'list' object is not callable

Why? Because max_value function’s first argument is list a Python builtin. Python allows to override them, but although could be useful in some really specific use cases, the general approach is to not do that as code then can suddenly break without a clear trace.

Example

Given the following code:

def my_method(object, list, dict):
    max = 5
    min = 3
    zip = (4, 3)

The following warnings are shown (via flake8):

test.py:1:15: B002 "object" is used as an argument and thus shadows a python builtin, consider renaming the argument
test.py:1:23: B002 "list" is used as an argument and thus shadows a python builtin, consider renaming the argument
test.py:1:29: B002 "dict" is used as an argument and thus shadows a python builtin, consider renaming the argument
test.py:2:5: B001 "max" is a python builtin and is being shadowed, consider renaming the variable
test.py:3:5: B001 "min" is a python builtin and is being shadowed, consider renaming the variable
test.py:4:5: B001 "zip" is a python builtin and is being shadowed, consider renaming the variable

Install

Install with pip:

$ pip install flake8-builtins

Requirements

  • Python 2.7, 3.3, 3.4

  • flake8

License

GPL 2.0

Changelog

0.1 (2016-03-04)

  • Initial release [gforcada]

  • Add buildout and other stuff. [gforcada]

  • Add actual code. [gforcada]

  • Drop support for python 3.3, only python 2.7 and python 3.4 are tested. [gforcada]

Project details


Download files

Download the file for your platform. If you're not sure which to choose, learn more about installing packages.

Source Distribution

flake8-builtins-0.1.tar.gz (15.0 kB view details)

Uploaded Source

File details

Details for the file flake8-builtins-0.1.tar.gz.

File metadata

File hashes

Hashes for flake8-builtins-0.1.tar.gz
Algorithm Hash digest
SHA256 1c93bf85d561d3ce91cb34959fd6d63aea74aefb2440230c6c55bdb3f7141f0c
MD5 1c4e282b345307112483ca5875c09cc5
BLAKE2b-256 593db2a99c2ff1a06c67c8f8003ed0365f2e25b0ce034a972dfce23261add8c1

See more details on using hashes here.

Supported by

AWS Cloud computing and Security Sponsor Datadog Monitoring Fastly CDN Google Download Analytics Pingdom Monitoring Sentry Error logging StatusPage Status page