Update CJS_PROXY_PREFIX

stealjs / steal-tools

Build easy. Load fast.

https://stealjs.com/docs/steal-tools.html

MIT License

66 stars 23 forks source link

Update CJS_PROXY_PREFIX #1129

Closed bmomberger-bitovi closed 5 years ago

bmomberger-bitovi commented 5 years ago

Rollup's CommonJS plugin changed the prefix that it uses for proxy files, since it was having problems with colons being used in file names.

This PR updates the prefix to match what the latest version of Rollup uses (colon replaced with hyphen).

Fixes #1125.

See https://github.com/rollup/rollup-plugin-commonjs/pull/371 for more info.

coveralls commented 5 years ago

Coverage remained the same at 93.322% when pulling 2694e42d97787dbb75eea756124fc98b4d5ebc92 on bmomberger-bitovi:1125-rollup-commonjs-prefix-change into 003ec65c5b61728d58ac0ff276d125a4ebbd839d on stealjs:master.