Skip to content

ruff format --exclude */source/conf.py#7458

Open
cclauss wants to merge 1 commit intoArduPilot:masterfrom
cclauss:ruff-format-without-source-conf.py
Open

ruff format --exclude */source/conf.py#7458
cclauss wants to merge 1 commit intoArduPilot:masterfrom
cclauss:ruff-format-without-source-conf.py

Conversation

@cclauss
Copy link
Contributor

@cclauss cclauss commented Mar 4, 2026

% ruff format --config="format.quote-style='preserve'" --exclude "*/source/conf.py"

The goal of this pull request is explained in the first paragraph of the psf/black docs:

By using Black, you agree to cede control over minutiae of hand-formatting. In return, Black gives you speed, determinism, and freedom from pycodestyle nagging about formatting. You will save time and mental energy for more important matters.

The change to .flake8 is to be compliant with PEP8.

@cclauss cclauss force-pushed the ruff-format-without-source-conf.py branch 2 times, most recently from 503afdc to 8bea003 Compare March 4, 2026 11:46
@Hwurzburg Hwurzburg requested review from khancyr and peterbarker and removed request for khancyr March 4, 2026 12:38
@khancyr
Copy link
Contributor

khancyr commented Mar 5, 2026

I want this ! But we should add either a pyproject.toml or ruff.toml to add the line-length = 127 as that what we are using currently and it will reduce the diff ! (black default to 88 ...)

@cclauss cclauss force-pushed the ruff-format-without-source-conf.py branch from 8bea003 to 121fc62 Compare March 6, 2026 18:07
@cclauss cclauss force-pushed the ruff-format-without-source-conf.py branch from 121fc62 to f2e27c1 Compare March 8, 2026 07:22
@cclauss cclauss force-pushed the ruff-format-without-source-conf.py branch 5 times, most recently from ae5b348 to 09c13ec Compare March 10, 2026 09:50
@cclauss cclauss force-pushed the ruff-format-without-source-conf.py branch from 09c13ec to 6b0262c Compare March 11, 2026 13:16
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.

2 participants