format behavior change

[a-frantz]

The current default behavior of sprocket format is to accept a single document, format it, and print the result to STDOUT. That isn't very useful in many contexts.

We should drop this behavior, and require either the --overwrite flag or the (new in #49 ) --check flag.

[Scott8440]

I'll also work on this one

[a-frantz]

Great! Before you get started, I just want to ping the rest of the team and ensure what this issue proposes (originally an idea from @adthrasher ) is exactly what we want. @claymcleod and @peterhuene haven't weighed in yet.

[claymcleod]

I think this is fine from a next step perspective. It's also relatively straightforward. I do think we should consider doing what cargo fmt does, which is make sure things are committed to source control before formatting. That being said, that's a bit more of an involved change, so I think we should go ahead with this to slightly improve things in the meantime.

[a-frantz]

I think this is fine from a next step perspective. It's also relatively straightforward. I do think we should consider doing what cargo fmt does, which is make sure things are committed to source control before formatting. That being said, that's a bit more of an involved change, so I think we should go ahead with this to slightly improve things in the meantime.

I agree. What's proposed in this issue is kind of a stop-gap to improve the UX immediately. It shouldn't be the behavior long-term, but it's good enough for now.

@Scott8440 you have the go-ahead to start work on this! Really appreciate your contributions :)

[claymcleod]

Closed in #51.