Skip to content

Run pycodestyle and pyflakes in CI#13

Merged
gordonmessmer merged 1 commit intogordonmessmer:mainfrom
hannob-forks:cicodingstyle
Feb 1, 2026
Merged

Run pycodestyle and pyflakes in CI#13
gordonmessmer merged 1 commit intogordonmessmer:mainfrom
hannob-forks:cicodingstyle

Conversation

@hannob
Copy link
Copy Markdown
Collaborator

@hannob hannob commented Jan 31, 2026

This adds some codingstyle checks to the CI.

It runs pycodestyle, which is the common tool to check for Python's PEP8 standard codingstyle (previously known as pep8).

I have disabled the following rules for now:

  • E501 - long lines, there are a couple in the code.
  • E722 - bare except. There are plenty in the code. Ideally, one would only catch specific exceptions, but this is often quite troublesome to implement as one has to know all the possible exceptions in advance.
  • W503/W504 - those two are incompatible, so you can only have one. They decide whether logical keywords (and/or) in a multiline if clause are on the next line or not. Both appear in the code, and there's no strong advantage of one over the other.

It also runs pyflakes, which detects unused variables and related issues.

@gordonmessmer gordonmessmer merged commit c9c3f12 into gordonmessmer:main Feb 1, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants