Python linting at Venmo

http://blog.venmo.com/2015/8/26/python-linting-at-venmo

by simon_weber • 11 years ago

6 2 11 years ago

blog.venmo.com

kolev• 11 years ago

Why everybody's doing their on hooks instead of using a framework such as pre-commit [0] or Overcommit [1]?

[0] http://pre-commit.com/

[1] https://github.com/brigade/overcommit

dalke• 11 years ago

Linters are indeed excellent tools.

A "what's wrong" which the linters don't find is that for/else is one of the more confusing aspects of the Python language, and should be avoided when there are equally clear alternatives.

That is, I believe that

    for entry in entries:
        if entry == key:
            break
    else:
        raise NotFoundError

is better written as:

    if not any(entry == key for entry in entries):
        raise NotFoundError

Python linting at Venmo

Related Stories