enb / enb-bemxjst

bem-xjst support for ENB

Not possible disable sourcemap

baymer opened this issue 5 years ago · comments

Mikhail Korotin commented 5 years ago

Because here https://github.com/enb/enb-bemxjst/blob/master/techs/bem-xjst.js#L142 not checks is the sourcemap enable or not (https://github.com/enb/enb-bemxjst/blob/master/techs/bem-xjst.js#L110).

Mikhail Korotin commented 5 years ago

I think fix is

if (this._sourcemap) {
  engineOptions.sourceMap = { from: this._target };
}

Vladimir Grinenko commented 5 years ago

@baymer, implemented just the way you suggested but with inverted condition to keep the change backward compatible.

Vladimir Grinenko commented 5 years ago

enb-bemxjst@8.10.6