Skip to content

Use ruff as formatter and linter#9

Merged
amgebauer merged 1 commit intoTUM-LNM:mainfrom
amgebauer:use-roff-as-formatter-and-linter
Aug 29, 2025
Merged

Use ruff as formatter and linter#9
amgebauer merged 1 commit intoTUM-LNM:mainfrom
amgebauer:use-roff-as-formatter-and-linter

Conversation

@amgebauer
Copy link
Contributor

No description provided.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR migrates the project's code formatting and linting from Black to Ruff, which is a faster Python linter and formatter written in Rust. The changes include updating the pre-commit configuration and reorganizing imports across all Python files to comply with Ruff's import sorting conventions.

  • Updated pre-commit configuration to use Ruff instead of Black for linting and formatting
  • Reorganized imports in all Python files to follow Ruff's import sorting standards (standard library, third-party, local imports)
  • Added Ruff configuration to ignore F401 (unused import) warnings in the main __init__.py file

Reviewed Changes

Copilot reviewed 19 out of 19 changed files in this pull request and generated 2 comments.

File Description
.pre-commit-config.yaml Replaced Black with Ruff for linting and formatting with specific rules
pyproject.toml Added Ruff configuration to ignore unused import warnings in init.py
src/biomesh/*.py Reorganized imports to follow Ruff's import sorting conventions
tests/*.py Reorganized imports to follow Ruff's import sorting conventions

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@amgebauer amgebauer force-pushed the use-roff-as-formatter-and-linter branch from f2b0166 to aec824b Compare August 29, 2025 15:08
@amgebauer amgebauer enabled auto-merge August 29, 2025 15:09
Copy link

@davidrudlstorfer davidrudlstorfer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice :)

@amgebauer amgebauer merged commit 7c8e932 into TUM-LNM:main Aug 29, 2025
7 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.

3 participants