go/hcl
2
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

fmtcmd: Remove extra newline

Browse Source 
The fmtcmd tests weren't running before due to the same build tag
problem as #112, so didn't catch the fact that there was an extra
newline being added here, which is now doubled thanks to #112.

Fixing up the build tag reveals failing tests - removing the extra
newline fixes the tests.
This commit is contained in:
Paul Hinze 2016-04-26 16:06:22 -05:00
parent 27a57f2605
commit 2f980537dd
No known key found for this signature in database
GPG Key ID: B69DEDF2D55501C0
2 changed files with 1 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
hcl/fmtcmd/fmtcmd.go
View File

@ -60,8 +60,6 @@ func processFile(filename string, in io.Reader, out io.Writer, stdin bool, opts
if err != nil { if err != nil {
return err return err
} }
// Files should end with newlines
res = append(res, []byte("\n")...)
if !bytes.Equal(src, res) { if !bytes.Equal(src, res) {
// formatting has changed // formatting has changed

2
hcl/fmtcmd/fmtcmd_test.go
View File

@ -1,4 +1,4 @@
// +build -windows // +build !windows
// TODO(jen20): These need fixing on Windows but fmt is not used right now // TODO(jen20): These need fixing on Windows but fmt is not used right now
// and red CI is making it harder to process other bugs, so ignore until // and red CI is making it harder to process other bugs, so ignore until
// we get around to fixing them. // we get around to fixing them.