commenthol/serialize-to-js

remove `beautify` option (`js-beautify` package too)

Closed this issue · 4 comments

alexander-akait commented

Why:

  • reduce package size
  • no need beautify by default, main purpose is serialization, not pretty printing
  • there a lot of other buitifiers (example i like prettier)
commenthol commented

you can use the serialization without beautifier.

const serialize = require('serialize-to-js/lib/serialize')

does the trick.

alexander-akait commented

@commenthol we want to use this library in big packages like loaders/plugins for webpack, and it is increase size our packages (node_modules), we don't want to do fork because of this problem, it is really looks weird

alexander-akait commented

Can we reconsider this issue, otherwise we really have to do fork?

commenthol commented

Hi @evilebottnawi,
there is PR #12 to split things up as you requested. Would be nice if you could have a look at it if that fits your/ and maybe others needs.