search

theia-ide / sprotty

A next-gen web-based graphics framework
Apache License 2.0
138 stars 23 forks source link

Upgraded to TypeScript 2.3.1 #141

Closed JanKoehnlein closed 7 years ago

JanKoehnlein commented 7 years ago

... and fixed a couple of type errors on the way

spoenemann commented 7 years ago

According to the npm documentation we shouldn't use an npm-shrinkwrap.json file:

It's strongly discouraged for library authors to publish this file, since that would prevent end users from having control over transitive dependency updates.

JanKoehnlein commented 7 years ago

What happened is that a minor update of typescript broke our tool for new users without us even noticing.

So what alternative do you propose instead? Using fixed version numbers? IMHO Shrinkwrap offers a decent workflow to gradually upgrade.