Remix.run Logo
nikisweeting 5 days ago

As long as it does `ruff check --fix; ruff format` internally and not just one or the other, then I'm happy. Tired of running 2 commands every time.

0cf8612b2e1e 5 days ago | parent | next [-]

Also needs to make sorting imports a default.

nikisweeting 5 days ago | parent [-]

and stop removing extra leading whitespace before inline comments (#7684)

zahlman 5 days ago | parent | prev [-]

FWIW, you can pretty easily script this sort of thing. In Bash it can even be a function added to your ~/.bashrc .

nikisweeting 5 days ago | parent | next [-]

Of course we already have it automated in pre-commit checks, I'm just mildly annoyed that it's two different commands in the first place (though I'm sure there is some good historical reason). The distinction/ordering between them isn't really clear as a user, I just want "find and fix everything you can, error if you cant autofix" every time.

IshKebab 5 days ago | parent | prev [-]

Just so you know, when people say "I hate this paper cut", they aren't really asking how they can put in extra work to get around it. Obviously hey can add some alias to ~/.bashrc... on every computer he uses. The complaint is that he has to do that in the first place. It's still annoying.

zahlman 5 days ago | parent [-]

I prefer having things this way, because it respects the fact that I may have a different idea from the next person about how the pieces should be put together.

aseipp 5 days ago | parent | next [-]

OK and the vast majority of people excluding you and them probably have a very close-or-nearly-identical idea about what should happen. So everyone can be happy, there's nothing being lost here.

wiseowise 4 days ago | parent | prev | next [-]

You can align it by hand, if you want.

nikisweeting 5 days ago | parent | prev [-]

no one is proposing removing the sub-commands, only adding a shortcut that calls both