splunk/qbec

Collect feedback on the new fmt command

gotwarlost opened this issue · 2 comments

Collect feedback on the new fmt command

from the outstanding items in the PR:

  • formatting files is general purpose and should probably not require a qbec root. We need to figure out what the behavior should be if someone types qbec alpha fmt --root /alt/root
  • add a JSON formatter as well
  • dump the full list of unformatted files rather than just the first one when using -e
  • better usability when user types something like qbec alpha fmt foobar.yaml (yaml formatting not turned on but requested for single yaml file. right now this is a confusing noop)
  • preserve blank lines in YAML formatting if at all possible

All of these are now resolved. Closing.