Conversation
Owner
Author
|
@cl3m3c7 Here I was using pylint, not ruff. Is it ok, or better to switch? |
Collaborator
|
@Mustard2 not sure 😅 https://docs.astral.sh/ruff/faq/#how-does-ruffs-linter-compare-to-pylint or both? |
we will use ruff for now, if it does not help at linting, we will add it back later.
Collaborator
|
I am working file by file basis, so it will take some time. go ahead ;) |
Refactor and format all the files based on Ruff
Collaborator
Owner
Author
|
Thank you very much for the huge work! Probably it would have been easier to review if it was split into different MR, but I guess it should be fine. I'll firstly merge 2026.4 so I have a version with the latest features to compare this one to. I can then try to use the UI a bit with this formatting to see if something breaks. Hopefully no, but it's too long to review in a standard way. :) |
Mustard2
commented
Apr 2, 2026
Mustard2
commented
Apr 3, 2026
Merged
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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.
See #232