Skip to content

Conversation

@aaron-skydio
Copy link
Member

format_py_dir doesn't work because it doesn't use the ruff config.

Example failure: https://github.com/symforce-org/symforce/actions/runs/20531137563/job/58982549124?pr=450

Here we have a line between 80 and 100 chars, so with the correct
config, it stays on the same line. If you format it with an 80-char
config and then a 100-char config though, it's broken across lines and
the trailing comma is added, so formatting with 100 chars doesn't put it
back on one line. And it has to be a list for the trailing comma thing
to happen, that's why this is pretty rare.

Topic: sf-ruff-wheel-types

format_py_dir doesn't work because it doesn't use the ruff config.

Example failure: https://github.com/symforce-org/symforce/actions/runs/20531137563/job/58982549124?pr=450

Here we have a line between 80 and 100 chars, so with the correct
config, it stays on the same line.  If you format it with an 80-char
config and then a 100-char config though, it's broken across lines and
the trailing comma is added, so formatting with 100 chars doesn't put it
back on one line.  And it has to be a list for the trailing comma thing
to happen, that's why this is pretty rare.

Topic: sf-ruff-wheel-types
@aaron-skydio
Copy link
Member Author

Reviews in this chain:
#452 Fix ruff format of types from wheel

@aaron-skydio
Copy link
Member Author

# head base diff date summary
0 ffd96ee3 f9420dc8 diff Dec 26 18:42 PM 2 files changed, 2 insertions(+), 16 deletions(-)

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