Clean rendered html output

[gael-boyenval]

Addresses #448

Summary of changes:

• add js-beautify dependency
• add clean into the render function

@bmuenzenmeyer
that's surely not the right place to do it. the tests are failing.
just for you too see.

maybe it would be more suited to modify pattern_assembler.renderPattern ?

[bmuenzenmeyer]

@gael-boyenval I think this could likely be moved to just before the writes.

Like around https://github.com/pattern-lab/patternlab-node/blob/master/core/lib/patternlab.js#L388-L395

[bmuenzenmeyer]

I pulled this down and have tested.
everything so far looks great! HTML is formatted better than nothing.

I see that the js-beautify library seems to accept an .editorconfig file via CLI and perhaps options. I wonder if we can get this to work via something like https://github.com/editorconfig/editorconfig-core-js

I didn't try it, but it'd be nice if the template code could be beautified too. It looks like you omitted that (because I presume, you don't know exactly which file we would be dealing with.) I think since most template languages are html-esque, it might work?

[gael-boyenval]

@bmuenzenmeyer : sorry i didn't saw th message earlier.
Regarding the templates, i think that the user has full hand on the output, since nothing is transformed, it's just copied.
I assumed that the user would want to keep them as they were created.
(assuming you are talking about pattern templates… yes ?)

[bmuenzenmeyer]

@gael-boyenval You'll see in my additional commit (now on dev!) that I am beautifying templates too.

I plan to release 2.6.0 as an alpha, so I think we can give it a go for a bit and see if there are any problems.

does that sound ok?

[bmuenzenmeyer]

sorry i didn't saw the message earlier.

and this was my first time using the new review interface, so I didn't know if it'd be the same as @mentioning you