Support STDIN/STDOUT

[alok]

If there was a flag like --stdout to output the changed text to STDOUT, that would be nice.

Being able to read from STDIN would make it perfect for autoformatters like neoformat.

[asottile]

I believe you're looking for --print-only?

[alok]

My bad. Does this support reading STDIN as well (AFK, so I can't check right now)?

[asottile]

reorder-python-imports /dev/stdin --print-only should work:

$ echo -e 'import os\nimport pre_commit\nimport sys' | reorder-python-imports /dev/stdin --print-only
==> /dev/stdin <==
import os
import sys

import pre_commit

[asottile]

(note that the ==> /dev/stdin <== output goes to stderr)

[asottile]

closing: if --print-only doesn't satisfy your usecase please provide additional context and I'll reopen :)

[alok]

Sorry, forgot about this. It needs to output specifically to STDOUT rather than STDERR since most editor formatter plugins only read STDOUT. Plus, it's not really an error as such, so it shouldn't be redirected there.

[asottile]

it does, you haven't tried it have you :P

[alok]

You're right, I didn't. I misread the sentence "(note that the ==> /dev/stdin <== output goes to stderr)". I thought you meant all of it was redirected to STDERR rather than just that literal line. My bad.

[alok]

On the other hand, I don't think Windows supports /dev/stdin. I don't use Windows, but I'd expect other people do.

[asottile]

If someone on windows wants the feature then we'll talk -- as it is it hasn't been an ask so far and if I'm not going to use it and you're not going to use it there's no point in complicating this tool

[pmiddend]

I'm trying to implement emacs support for this fine tool. This would be a little easier if --print-only always printed the result, whether there was a reordering or not. Is that possible?