jasonkuhrt / react-popover

A smart popover component for React
600 stars 240 forks source link

babel-jest included as production dependency #115

Closed mjilugu closed 7 years ago

mjilugu commented 7 years ago

I noticed react-popover having a huge dependency tree in production. Looks like babel-jest could move to devDependencies.

https://github.com/facebook/jest/tree/master/packages/babel-jest

├─┬ react-popover@0.4.9 
│ ├─┬ babel-jest@19.0.0 
│ │ ├─┬ babel-plugin-istanbul@4.1.4 
│ │ │ ├─┬ find-up@2.1.0 
│ │ │ │ └─┬ locate-path@2.0.0 
│ │ │ │   ├─┬ p-locate@2.0.0 
│ │ │ │   │ └── p-limit@1.1.0 
│ │ │ │   └── path-exists@3.0.0 
│ │ │ ├─┬ istanbul-lib-instrument@1.7.2 
│ │ │ │ ├── istanbul-lib-coverage@1.1.1 
│ │ │ │ └── semver@5.3.0 
│ │ │ └─┬ test-exclude@4.1.1 
│ │ │   ├── arrify@1.0.1 
│ │ │   ├─┬ micromatch@2.3.11 
│ │ │   │ ├─┬ arr-diff@2.0.0 
│ │ │   │ │ └── arr-flatten@1.0.3 
│ │ │   │ ├── array-unique@0.2.1 
│ │ │   │ ├─┬ braces@1.8.5 
│ │ │   │ │ ├─┬ expand-range@1.8.2 
│ │ │   │ │ │ └─┬ fill-range@2.2.3 
│ │ │   │ │ │   ├── is-number@2.1.0 
│ │ │   │ │ │   ├── isobject@2.1.0 
│ │ │   │ │ │   ├── randomatic@1.1.6 
│ │ │   │ │ │   └── repeat-string@1.6.1 
│ │ │   │ │ ├── preserve@0.2.0 
│ │ │   │ │ └── repeat-element@1.1.2 
│ │ │   │ ├─┬ expand-brackets@0.1.5 
│ │ │   │ │ └── is-posix-bracket@0.1.1 
│ │ │   │ ├── extglob@0.3.2 
│ │ │   │ ├── filename-regex@2.0.1 
│ │ │   │ ├── is-extglob@1.0.0 
│ │ │   │ ├── is-glob@2.0.1 
│ │ │   │ ├─┬ kind-of@3.2.2 
│ │ │   │ │ └── is-buffer@1.1.5 
│ │ │   │ ├─┬ normalize-path@2.1.1 
│ │ │   │ │ └── remove-trailing-separator@1.0.1 
│ │ │   │ ├─┬ object.omit@2.0.1 
│ │ │   │ │ ├─┬ for-own@0.1.5 
│ │ │   │ │ │ └── for-in@1.0.2 
│ │ │   │ │ └── is-extendable@0.1.1 
│ │ │   │ ├─┬ parse-glob@3.0.4 
│ │ │   │ │ ├─┬ glob-base@0.3.0 
│ │ │   │ │ │ └── glob-parent@2.0.0 
│ │ │   │ │ └── is-dotfile@1.0.2 
│ │ │   │ └─┬ regex-cache@0.4.3 
│ │ │   │   ├── is-equal-shallow@0.1.3 
│ │ │   │   └── is-primitive@2.0.0 
│ │ │   ├─┬ read-pkg-up@1.0.1 
│ │ │   │ ├─┬ find-up@1.1.2 
│ │ │   │ │ ├── path-exists@2.1.0 
│ │ │   │ │ └─┬ pinkie-promise@2.0.1 
│ │ │   │ │   └── pinkie@2.0.4 
│ │ │   │ └─┬ read-pkg@1.1.0 
│ │ │   │   ├─┬ load-json-file@1.1.0 
│ │ │   │   │ ├── graceful-fs@4.1.11 
│ │ │   │   │ ├─┬ parse-json@2.2.0 
│ │ │   │   │ │ └─┬ error-ex@1.3.1 
│ │ │   │   │ │   └── is-arrayish@0.2.1 
│ │ │   │   │ ├── pify@2.3.0 
│ │ │   │   │ └─┬ strip-bom@2.0.0 
│ │ │   │   │   └── is-utf8@0.2.1 
│ │ │   │   ├─┬ normalize-package-data@2.3.8 
│ │ │   │   │ ├── hosted-git-info@2.4.2 
│ │ │   │   │ ├─┬ is-builtin-module@1.0.0 
│ │ │   │   │ │ └── builtin-modules@1.1.1 
│ │ │   │   │ └─┬ validate-npm-package-license@3.0.1 
│ │ │   │   │   ├─┬ spdx-correct@1.0.2 
│ │ │   │   │   │ └── spdx-license-ids@1.2.2 
│ │ │   │   │   └── spdx-expression-parse@1.0.4 
│ │ │   │   └── path-type@1.1.0 
│ │ │   └── require-main-filename@1.0.1 
│ │ └─┬ babel-preset-jest@19.0.0 
│ │   └── babel-plugin-jest-hoist@19.0.0 
│ ├── create-react-class@15.5.3 
│ ├─┬ css-vendor@0.3.8 
│ │ └── is-in-browser@1.1.3 
│ ├─┬ lodash.throttle@3.0.4 
│ │ └─┬ lodash.debounce@3.1.1 
│ │   └── lodash._getnative@3.9.1 
│ └── prop-types@15.5.10 
jasonkuhrt commented 7 years ago

@mjilugu Thanks for reporting. Will fix this tonight!

mjilugu commented 7 years ago

👍 Thanks a mil!