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

Python Linter #102

Open
aaron97neu opened this issue Oct 9, 2020 · 2 comments
Open

Python Linter #102

aaron97neu opened this issue Oct 9, 2020 · 2 comments
Labels
enhancement New feature or request

Comments

@aaron97neu
Copy link
Member

We are starting to write a large amount of python using varying styles. We should start using a linter to enforce formatting and stylistic guidelines

Pylint seems be the defacto linter. By default it enforces PEP8 which is the official standard. I don't necessarily agree with all of PEP8's standards and we can customize it with a pylintrc, however we should keep that to a minimum

Is it possible to include this with our CI stuff as well? That might be cool

@aaron97neu aaron97neu added the enhancement New feature or request label Oct 9, 2020
@abalmos
Copy link
Member

abalmos commented Oct 9, 2020

Agreed. I personally prefer to use auto formaters (autopep8 in this case) and just stop having format arguments. Lint rules would be a great add. We can create tasks to run pylint for sure.

@facastiblancor
Copy link
Collaborator

I agree too. That way we have consistency across all scripts. I am also in favor of lint rules.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants