ckeditor / ckeditor5-dev / 4655 / 1

90%

master: 90%

push

travis-ci-com

Merge pull request #833 from ckeditor/ck/13136

Other (utils): The `getDLLPluginWebpackConfig()` function requires webpack as its first argument. Closes ckeditor/ckeditor5#13136.

Other (utils): Removed `webpack` from peer dependencies.

MAJOR BREAKING CHANGE (utils): The `getDLLPluginWebpackConfig()` does not load webpack manually. Now it must be specified in arguments. 

Before:

```js
const { builds } = require( '@ckeditor/ckeditor5-dev-utils' );

builds.getDllPluginWebpackConfig( { /* options */ } );
```

After:

```js
const { builds } = require( '@ckeditor/ckeditor5-dev-utils' );
const webpack = require( 'webpack' );

builds.getDllPluginWebpackConfig( webpack, { /* options */ } );
```

1297 of 1479 branches covered (87.69%)

Branch coverage included in aggregate %.

2974 of 3284 relevant lines covered (90.56%)

31.48 hits per line