@Baughn, thanks for your PR! By analyzing the history of the files in this pull request, we identified @wlhlm, @globin and @domenkozar to be potential reviewers.

Example output: http://sprunge.us/WKCc

Nice! What about removing some blank lines, like sed 'N; /^\s*\n$/d; P; D' ?

I feel like that should be done in the formatter, not with sed scripts. I like my config files to have some blank lines, and think the current output is probably better than removing all of them.

But in reality, I'm unlikely to write any code to fix that. The config file is readable now.

You are probably right here. nginfmt should remove extra lines (quote, neighbouring empty lines are collapsed to at most two empty lines)

What still needs doing before this can be merged?

perhaps a confirmation from @globin

I haven't had the time to test this and won't have until the weekend, but the code looks good so if anyone confirms he has tested this I'd be very happy to merge :)

I've tested it, of course... guess that doesn't count. :P

I've tested this too and already use on my servers