Skip to content

Get rid of unnecessary dependencies #2113

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
Awk34 opened this issue Aug 4, 2016 · 2 comments · Fixed by #2122
Closed

Get rid of unnecessary dependencies #2113

Awk34 opened this issue Aug 4, 2016 · 2 comments · Fixed by #2122
Assignees
Milestone

Comments

@Awk34
Copy link
Member

Awk34 commented Aug 4, 2016

We should remove all the now-unnecessary dependencies from the package.json before the 4.0.0 release

@Awk34 Awk34 added this to the 4.0.0 milestone Aug 4, 2016
@Awk34 Awk34 assigned Koslun and Awk34 Aug 4, 2016
@Koslun
Copy link
Member

Koslun commented Aug 4, 2016

Can create a PR later in the week but here are some of them:

gulp-ng-annotate
gulp-ng-constant
gulp-typescript
gulp-uglify

@Awk34
Copy link
Member Author

Awk34 commented Aug 4, 2016

depcheck Shows me this on a default install:

$ depcheck
Unused dependencies
* bootstrap-sass
* bootstrap
* bootstrap-social
* angular-animate
* angular-aria
* font-awesome
* angular-validation-match
* babel-runtime
* socketio-jwt
Unused devDependencies
* bs-fullscreen-message
* babel-plugin-transform-runtime
* eslint
* gulp-add-src
* gulp-angular-templatecache
* gulp-autoprefixer
* gulp-cache
* gulp-concat
* gulp-filter
* gulp-clean-css
* gulp-ng-annotate
* gulp-rename
* gulp-svgmin
* gulp-uglify
* gulp-useref
* gulp-sass
* gulp-scss-lint
* utile
* webpack-dev-server
* ng-annotate-loader
* babel-loader
* css-loader
* file-loader
* imports-loader
* isparta-instrumenter-loader
* isparta-loader
* istanbul-instrumenter-loader
* null-loader
* postcss-loader
* raw-loader
* style-loader
* sass-loader
* tiny-lr
* istanbul

There's a lot in there I know we do use, but at least this provides a starting point

Awk34 added a commit that referenced this issue Aug 5, 2016
Awk34 added a commit that referenced this issue Aug 5, 2016
@Awk34 Awk34 closed this as completed Aug 7, 2016
Awk34 added a commit that referenced this issue Aug 7, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants