Skip to content

Update CJS_PROXY_PREFIX#1129

Merged
matthewp merged 1 commit intostealjs:masterfrom
bmomberger-bitovi:1125-rollup-commonjs-prefix-change
Apr 16, 2019
Merged

Update CJS_PROXY_PREFIX#1129
matthewp merged 1 commit intostealjs:masterfrom
bmomberger-bitovi:1125-rollup-commonjs-prefix-change

Conversation

@bmomberger-bitovi
Copy link
Contributor

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 rollup/rollup-plugin-commonjs#371 for more info.

@coveralls
Copy link

Coverage Status

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

@matthewp matthewp merged commit ec17ec2 into stealjs:master Apr 16, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fix integration with rollup-plugin-commonjs

3 participants