fixes

[juliocbcotta]

• fixes dep versions to avoid updates breaking stuff. (this is what happened with tslint in 1412b82 ); and closes would be nice to upgrade versions to latest #16
• closes compilation errors #14

[juliocbcotta]

@mmolhoek, those are not direct dependencies, like lodash is a angular dependencies, as far as I know, we need to wait them update, so we update the angular version.

[lathonez]

yep

On 25 February 2016 at 09:58, Júlio Cesar Bueno Cotta <
[email protected]> wrote:

@mmolhoek https://github.com/mmolhoek, those are not direct
dependencies, like lodash is a angular dependencies, as far as I know, we
need to wait them update, so we update the angular version.

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

[mmolhoek]

@BugsBunnyBR ok 👍

[juliocbcotta]

@lathonez , I don't know if it solves anything. I just forgot to remove that webpack lines.

[mmolhoek]

@lathonez @BugsBunnyBR it did remove a ton of warning on npm start:

ERROR in [default] /Volumes/SecureProjects/Projects/angular2/clicker/typings/main/ambient/jasmine/jasmine.d.ts:473:8
Duplicate identifier 'started'.

ERROR in [default] /Volumes/SecureProjects/Projects/angular2/clicker/typings/main/ambient/jasmine/jasmine.d.ts:474:8
Duplicate identifier 'finished'.

ERROR in [default] /Volumes/SecureProjects/Projects/angular2/clicker/typings/main/ambient/jasmine/jasmine.d.ts:475:8
Duplicate identifier 'result'.

ERROR in [default] /Volumes/SecureProjects/Projects/angular2/clicker/typings/main/ambient/jasmine/jasmine.d.ts:476:8
Duplicate identifier 'messages'.

ERROR in [default] /Volumes/SecureProjects/Projects/angular2/clicker/typings/main/ambient/jasmine/jasmine.d.ts:490:8
Duplicate identifier 'Spec'.

ERROR in [default] /Volumes/SecureProjects/Projects/angular2/clicker/typings/main/ambient/jasmine/jasmine.d.ts:491:8
Duplicate identifier 'clock'.

ERROR in [default] /Volumes/SecureProjects/Projects/angular2/clicker/typings/main/ambient/jasmine/jasmine.d.ts:492:8
Duplicate identifier 'util'.

[lathonez]

@mmolhoek this will be the tsconfig.json change?

[mmolhoek]

@lathonez yep

[juliocbcotta]

@lathonez Yes, it will, for now.
@mmolhoek , the remaining lines need ionic and angular update, which are not available.

[lathonez]

http://stackoverflow.com/questions/22343224/difference-between-tilde-and-caret-in-package-json

(for my own reference)

[lathonez]

merged as 90f677d

This was sufficient to resolve all the warnings for me.

@BugsBunnyBR - thanks a lot for picking this up, will teach me to actually start the app when I make changes like this (instead of just testing). Hopefully when Ionic moves back to gulp we can use an identical compilation strategy for both.

@mmolhoek - cheers for your input on this and the other issues.