cyclejs-community / create-cycle-app

Create Cycle.js apps with no build configuration.
ISC License
238 stars 21 forks source link

Duplicate versions of xstream #117

Closed geovanisouza92 closed 6 years ago

geovanisouza92 commented 7 years ago

Code to reproduce the issue:

create-cycle-app example --flavor cycle-scripts-one-fits-all
cd example
npm run eject
npm install --save-dev webpack-bundle-analyzer

Change webpack.config.js as the following;

const BundleAnalyzerPlugin = require('webpack-bundle-analyzer').BundleAnalyzerPlugin;

module.exports = {
    // ...
    plugins: [new BundleAnalyzerPlugin()]
};
npm run build

Expected behavior:

Include xstream on the bundle only once (the blue rect)

Actual behavior:

Included four times on the bundle

selecao_007

Versions of packages used:

yarn list v0.27.5
├─ @cycle/dom@18.3.0
├─ @cycle/http@14.4.0
├─ @cycle/isolate@3.1.0
├─ @cycle/run@3.3.0
├─ @cycle/time@0.8.0
├─ cycle-onionify@3.3.0
├─ cycle-restart@0.2.2
├─ xstream@10.9.0

This is related to @cycle/run, @cycle/time and cycle-onionify, spotted on https://github.com/cyclejs/cyclejs/issues/700

jvanbruegge commented 7 years ago

Will investigate tomorrow Together with this uglify bug

jvanbruegge commented 6 years ago

Not the case any more in one-fits-all V5