Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

pydrake lint: Should add and fix flake8 --select=F checks #14234

Closed
EricCousineau-TRI opened this issue Oct 21, 2020 · 1 comment
Closed

pydrake lint: Should add and fix flake8 --select=F checks #14234

EricCousineau-TRI opened this issue Oct 21, 2020 · 1 comment
Assignees
Labels
component: pydrake Python API and its supporting Starlark macros priority: low type: cleanup

Comments

@EricCousineau-TRI
Copy link
Contributor

On 6fa365e, we gots a few:

$ find bindings/pydrake -name '*.py' | sort | xargs flake8 --select F bindings/pydrake/**/*.py | grep -v '\*' | wc -l
424

Relates ye olde #2465

@EricCousineau-TRI EricCousineau-TRI self-assigned this Oct 21, 2020
@EricCousineau-TRI EricCousineau-TRI changed the title pydrake lint: Should add flake8 --select=F checks pydrake lint: Should add and fix flake8 --select=F checks Oct 21, 2020
@ggould-tri ggould-tri added component: pydrake Python API and its supporting Starlark macros unused team: manipulation type: cleanup labels Oct 22, 2020
@jwnimmer-tri
Copy link
Collaborator

Closing as duplicate of #19827.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component: pydrake Python API and its supporting Starlark macros priority: low type: cleanup
Projects
None yet
Development

No branches or pull requests

3 participants