Fix trivial linter warnings and errors #13
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
import, move the registration into a dedicated function
… deprecated `List`
Only the debugging helper need to be imported and made available by default.
external file included via `!include`
This is likely an unattentive copy and paste error from the pyyaml documentation.
This is likely an unattentive copy and paste error from the pyyaml documentation.
This is likely an unattentive copy and paste error from the pyyaml documentation.
This is likely an unattentive copy and paste error from the pyyaml documentation.
This is likely an unattentive copy and paste error from the pyyaml documentation.
This is likely an unattentive copy and paste error from the pyyaml documentation.
variable `debug` and the its associated `debug_print` function
|
I've merged this now, even though there are still more things the linter complains about. Since I'm working on two other PRs at the moment, I want to minimize the divergence between them, thus the remaining linter issues will be resolved after the open PRs have been completed. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This fixes some of the trivial linter warnings & errors that are reported by ruff and pylint.
None of these changes are functionally relevant, no bugs have been found.
There are still quite a few warnings left, a few just require the proper configuration of ruff and pylint.
Some of the remaining warnings & errors require more invasive changes and will be addressed in separate PRs.