You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Congrats on the release with the new formatter. Besides a few changed lines that I still need to investigate It looks really good!
I was a bit surprised with the fact that it is only a separate command. I kind of expected it to work as a lint rule: RUF300: Code not formatted or something... Which of course includes an autofix.
Using Ruff would allow me to drop 3 tools (flake8, isort, black) but I still need to run two commands...
I also noticed #8185 and feel that those warnings are better suited in the case that the formatter is enabled as lint rule. It seems to me that as a separate command you might want to differ between formatting and conflicting lint rules...
The text was updated successfully, but these errors were encountered:
I'm going to close this issue in favor of a tracking issue at #8232 which we'll use for general updates on this feature — implementing as a lint rule is only one of the possible interfaces.
Congrats on the release with the new formatter. Besides a few changed lines that I still need to investigate It looks really good!
I was a bit surprised with the fact that it is only a separate command. I kind of expected it to work as a lint rule:
RUF300: Code not formatted
or something... Which of course includes an autofix.Using Ruff would allow me to drop 3 tools (flake8, isort, black) but I still need to run two commands...
I also noticed #8185 and feel that those warnings are better suited in the case that the formatter is enabled as lint rule. It seems to me that as a separate command you might want to differ between formatting and conflicting lint rules...
The text was updated successfully, but these errors were encountered: