Repository navigation
refactor: sort dependencies alphabetically - #5090
BernhardRode wants to merge 3 commits into
Conversation
Sorted dependencies makes merging a lot easier. Helps a lot, when upgrading the cli and pulling in the correct deps.
| "@angular/http": "<%= ng4 ? '>=4.0.0-beta <5.0.0' : '^2.4.0' %>", | ||
| "@angular/platform-browser": "<%= ng4 ? '>=4.0.0-beta <5.0.0' : '^2.4.0' %>", | ||
| "@angular/platform-browser-dynamic": "<%= ng4 ? '>=4.0.0-beta <5.0.0' : '^2.4.0' %>", | ||
| "@angular/platform-browser": "<%= ng4 ? '>=4.0.0-beta <5.0.0' : '^2.4.0' %>", |
There was a problem hiding this comment.
npm install will flip the order of these two. @angular/platform-browser should be first.
| "karma-coverage-istanbul-reporter": "^0.2.0", | ||
| "karma-jasmine-html-reporter": "^0.2.2", | ||
| "karma-jasmine": "~1.1.0", | ||
| "karma": "~1.4.1", |
There was a problem hiding this comment.
npm install will flip the order of these aswell. Shortest name first.
|
I just use yarn and a sort alphabetical Plugin inside of my editor. |
|
You should have rebased, not merge master. And the only package I can see that is not in alphabetic order is "karma-coverage-istanbul-reporter" (comes from #4560) and it's still not in the correct place. |
|
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
Sorted dependencies makes merging a lot easier.
Helps a lot, when upgrading the cli and pulling in the correct deps.