[Request] Change `-in` to `-`

Question

[Request] Change `-in` to `-`

Suavesito-Olimpiada opened this issue 2 years ago · comments

First of, thanks for the amazing tool! :D

In many CLI programs is usual that stdin is configured automatically for input if no arguments were added. But, in many others (were no arguments means print help) it is (almost) standard to have - mean read from stdin.

I know is not important, but I propose follow the semantics of others, as I think allow for a (a little bit) more seamless experience. :)

Braydon Kains · Answer 1 · Wed Aug 24 2022 21:34:36 GMT+0800 (China Standard Time)

Thank you for the suggestion! It didn't occur to me during development to do this, but it definitely makes sense. I'll target it for v0.3.0.

Braydon Kains · Answer 2 · Thu Aug 25 2022 04:05:30 GMT+0800 (China Standard Time)

@Suavesito-Olimpiada I opened a PR for this change. Please let me know if this matches what you had in mind!
I had to do it a bit weirdly to keep consistent behaviour on Windows, since I assume - wouldn't be an alias to /dev/stdin on that.

Suavesito · Answer 3 · Thu Aug 25 2022 05:48:28 GMT+0800 (China Standard Time)

Yeah, that looks exactly like it. I understand why you make it that way, /dev/stdin does not exist properly on Windows and currently yamlfmt will not output anything if passed /dev/stdin on Linux (although it "works" in the sense that it does not crash).