Skip to content
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

build: npm dependencies update + dedupe #6213

Closed
wants to merge 31 commits into from

Conversation

IgorMinar
Copy link
Contributor

This PR absorbed #6067

The goal of this PR is to update our dependencies and dedupe them to take advantage npm3's features and decrease the installation time.

Current state:

$ npm outdated

Package  Current   Wanted   Latest
  • broccoli-funnel 0.2.3 git git
  • angular 1.4.7 1.4.8 1.4.8
  • angular-animate 1.4.7 1.4.8 1.4.8
  • angular-mocks 1.4.7 1.4.8 1.4.8
  • bower 1.6.5 1.7.2 1.7.2
  • broccoli 0.15.4 0.15.4 0.16.9 -
  • broccoli-funnel 0.2.3 git git
  • broccoli-slow-trees 1.1.0 1.1.0 2.0.0
  • broccoli-stew 0.2.1 0.2.1 1.2.0
  • browserify 10.2.6 10.2.6 12.0.1
  • chokidar 1.2.0 1.4.2 1.4.2
  • clang-format 1.0.32 1.0.33 1.0.33
  • conventional-changelog 0.2.1 0.2.1 0.5.3
  • del 1.2.1 1.2.1 2.2.0
  • es6-shim 0.33.10 0.33.13 0.34.0
  • fs-extra 0.18.4 0.18.4 0.26.3
  • glob 4.5.3 4.5.3 6.0.3
  • googleapis 1.0.23 1.0.23 2.1.7
  • gulp-connect 1.1.1 1.1.1 2.3.1
  • gulp-load-plugins 1.0.0 1.1.0 1.1.0
  • gulp-sass 2.1.0 2.1.1 2.1.1
  • gulp-tslint 3.5.0 3.6.0 4.2.2
  • gulp-uglify 1.4.2 1.5.1 1.5.1
  • jasmine 2.3.1 2.3.1 2.4.1
  • jpm 1.0.0 1.0.0 1.0.4
  • js-yaml 3.4.3 3.4.6 3.4.6
  • karma 0.13.15 0.13.16 0.13.16 - blocked on Karma chrashes with: Fatal error: Wrong length [0.13.16 regression] karma-runner/karma#1768
  • karma-browserstack-launcher 0.1.7 0.1.8 0.1.8
  • karma-chrome-launcher 0.2.1 0.2.2 0.2.2
  • karma-sauce-launcher 0.2.14 0.2.14 0.3.0
  • lodash 2.4.2 2.4.2 3.10.1
  • madge 0.5.0 0.5.0 0.5.3 - blocked on Circular dependency issues #5022
  • minimatch 2.0.10 2.0.10 3.0.0
  • minimist 1.1.3 1.1.3 1.2.0
  • mock-fs 2.7.1 2.7.1 3.6.0
  • node-uuid 1.4.3 1.4.7 1.4.7
  • parse5 1.3.2 1.3.2 2.0.2
  • protractor 2.5.1 2.5.1 3.0.0
  • proxy-middleware 0.13.1 0.13.1 0.15.0
  • react 0.14.2 0.14.5 0.14.5
  • rewire 2.3.4 2.5.1 2.5.1
  • run-sequence 1.1.4 1.1.5 1.1.5
  • semver 4.3.6 4.3.6 5.1.0
  • source-map 0.3.0 0.3.0 0.5.3
  • strip-ansi 2.0.1 2.0.1 3.0.0
  • systemjs 0.18.10 0.18.10 0.19.9
  • systemjs-builder 0.10.6 0.10.6 0.14.15
  • through2 0.6.5 0.6.5 2.0.0
  • ts2dart 0.7.18 0.7.19 0.7.19
  • universal-analytics 0.3.9 0.3.10 0.3.10
  • webpack 1.12.6 1.12.9 1.12.9
  • which 1.2.0 1.2.1 1.2.1
  • yargs 2.3.0 2.3.0 3.31.0

@googlebot
Copy link

We found a Contributor License Agreement for you (the sender of this pull request) and all commit authors, but as best as we can tell these commits were authored by someone else. If that's the case, please add them to this pull request and have them confirm that they're okay with these commits being contributed to Google. If we're mistaken and you did author these commits, just reply here to confirm.

@IgorMinar IgorMinar mentioned this pull request Jan 3, 2016
7 tasks
@IgorMinar IgorMinar added the action: review The PR is still awaiting reviews from at least one requested reviewer label Jan 3, 2016
@IgorMinar IgorMinar added this to the beta.1 milestone Jan 3, 2016
@IgorMinar IgorMinar added cla: yes and removed cla: no labels Jan 3, 2016
@googlebot googlebot added cla: no and removed cla: yes labels Jan 5, 2016
@IgorMinar IgorMinar force-pushed the npm-deps-update branch 2 times, most recently from c6517cb to 104dff8 Compare January 12, 2016 07:02
@IgorMinar IgorMinar force-pushed the npm-deps-update branch 3 times, most recently from 711cd67 to 82ecd48 Compare January 20, 2016 19:36
@IgorMinar IgorMinar changed the title WIP: npm dependencies update + dedupe build: npm dependencies update + dedupe Jan 21, 2016
@btford
Copy link
Contributor

btford commented Jan 21, 2016

LGTM

@IgorMinar IgorMinar added pr_state: LGTM action: merge The PR is ready for merge by the caretaker labels Jan 21, 2016
@IgorMinar
Copy link
Contributor Author

CI is flaking out, rerunning the job now. Circle CI failure is a flake as well. We should be ignored for now - I'll fix that in a separate PR.

@rkirov
Copy link
Contributor

rkirov commented Jan 21, 2016

How about squashing it, before merging? This many commits (31) will dilute our history.

@IgorMinar
Copy link
Contributor Author

Lots not squash in case we need to bisect an issue. I've done as much
testing as I could but there are quite a few changes here and squashing
them will make any follow up debugging hard.
On Thu, Jan 21, 2016 at 3:25 PM Rado Kirov notifications@github.com wrote:

How about squashing it, before merging? This many commits (31) will dilute
our history.


Reply to this email directly or view it on GitHub
#6213 (comment).

@mary-poppins
Copy link

Merging PR #6213 on behalf of @rkirov to branch presubmit-rkirov-pr-6213.

@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Sep 7, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
action: merge The PR is ready for merge by the caretaker action: review The PR is still awaiting reviews from at least one requested reviewer cla: yes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants