Formatting files using shfmt results in an empty file when shfmt can't parse the file

[chris-reeves]

When formatting is enabled and run against a file that shfmt can't parse, the contents are replaced with an empty file. This is a result of improper handling when shfmt returns a non-zero exit status.

Originally reported by @David-Else in #1136 (comment)