-
Would it be worth adding some CI workflows to add checks to PRs? I feel like it would help to ensure the code conforms to the same standard of quality across the repo, and would make sure no broken code is merged accidentally. |
Beta Was this translation helpful? Give feedback.
Replies: 2 comments 1 reply
-
To extend this, tools I generally apply to this task are Black, Pylint, Flake8 and Isort. These can be configured to work nicely together, or you can pick 1/2 tools for the job. I generally quite like the Black style guidelines, and I think it makes for very neat and readable code. |
Beta Was this translation helpful? Give feedback.
-
I'm not really a fan of linters in Python since they're noisy. Likewise, I don't like black in its default configuration since it's too opinionated. This has been brought up before, see here but currently I don't really want it. The only type of automation I'd want would probably be checking for extraneous whitespace but that's a different matter than what is proposed here. |
Beta Was this translation helpful? Give feedback.
I'm not really a fan of linters in Python since they're noisy. Likewise, I don't like black in its default configuration since it's too opinionated. This has been brought up before, see here but currently I don't really want it. The only type of automation I'd want would probably be checking for extraneous whitespace but that's a different matter than what is proposed here.