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.

Sign up for GitHub

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

Jump to bottom

Install a python linter #120

Open
subzero10 opened this issue Sep 30, 2022 · 4 comments
Open

Install a python linter #120

subzero10 opened this issue Sep 30, 2022 · 4 comments

Comments

@subzero10
Copy link
Member

My Intellij IDE complains with style warnings and inconsistencies with PEP 8.
I think we should integrate python linter and include in our CI.
A quick google search brought up the following linters:

@joshuap Do you have any experience/preference towards any of the above?
@joshuap
Copy link
Member

joshuap commented Oct 3, 2022

I don't have a personal preference—what about you, @Kelvin4664? Otherwise I'd go with whichever seems more popular.

@Kelvin4664
Copy link
Collaborator

No personal preference either. I am good with either of both.

@dotysan
Copy link
Contributor

dotysan commented Jan 21, 2023

Start with flake8. You can always add others later.

@dotysan
Copy link
Contributor

dotysan commented Jun 4, 2023

I took an initial stab in PR #149 and cleaning up.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@joshuap @dotysan @subzero10 @Kelvin4664