Conversation
|
All I did was run |
fix ci fix flake8
|
For some reason flake8 on the CI is not happy with some changes, not sure why it differs for your local run. There is also the very similar #690 though, which would move from flake8 to ruff. It needs some rebasing currently. @lazka What's your take on this? There seems to be quite some demand of modernizing the code base by consistent formatting and adding type hints. There are several issues and PRs that touch this, and I think it totally makes sense. We recently moved Picard's code formatting to ruff, and I'm very satisfied with it. Would be my personal preference. For type hinting I think having at least the public API part type hinted also will be very useful. I'm not much of a fan of overdoing the type hints in Python, but I have noticed we have been using this more and more in Picard as well whenever we touched parts of the code. |
|
that's likely because I bumped the mypy version and made it more strict. I can revert those if you want to make CI happy before accepting the PR |
|
sorry to bug you again. have you looked into this at all? |
I have changed the PR to be smaller. Now all it does is
pyupgradeto at least Python 3.12.All tests passed on my Windows 11 pc.
Old PR #700