Skip to content

Add Ruff#743

Open
CommandMC wants to merge 3 commits into
legendary-gl:masterfrom
CommandMC:not-beating-the-puppygirl-allegations
Open

Add Ruff#743
CommandMC wants to merge 3 commits into
legendary-gl:masterfrom
CommandMC:not-beating-the-puppygirl-allegations

Conversation

@CommandMC

@CommandMC CommandMC commented Apr 9, 2026

Copy link
Copy Markdown
Member

Ruff is a very fast linter & formatter. I figured we could use it to have a cleaner style guide, since contributions are now coming from multiple people.

The linting rules I've added so far are rather lax on purpose, I want to gradually add more in the future. To start with, I want to remove most of the ignored rules again.
None of the changes made to comply with them affect functionality. They boil down to:

  • Changing f-strings to normal strings if there are no placeholders
  • Removing # coding: utf-8 from the start of almost every file
  • Not using star-imports
  • Sorting imports
  • Renaming/removing unused variables

Formatting was not done at all yet, I wanted to establish a style guide first before modifying basically everything.

@CommandMC CommandMC self-assigned this Apr 9, 2026
@CommandMC CommandMC force-pushed the not-beating-the-puppygirl-allegations branch from 1b17604 to 301b897 Compare April 9, 2026 17:40
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.

3 participants