panuhorsmalahti / gulp-tslint

TypeScript linter plugin for Gulp
MIT License
118 stars 44 forks source link

@types should be in devDependencies #126

Closed jbedard closed 7 years ago

jbedard commented 7 years ago

Since 8.1 some type definitions were added to dependencies. I think these should be in the devDependencies so they don't get installed along with gulp-tslint?

panuhorsmalahti commented 7 years ago

Fixed in 8.1.1.

demurgos commented 6 years ago

Types with global definitions (environment/globals) should be installed by the consumer, but all the other types should be regular dependencies so type definitions work reliably. In practice, it means that types for normal dependencies should also be normal dependencies.