dialogflow-javascript-client icon indicating copy to clipboard operation
dialogflow-javascript-client copied to clipboard

Publish .js and .d.ts files on npmjs instead of .ts.

Open drewsilcock opened this issue 8 years ago • 1 comments

See https://github.com/Microsoft/TypeScript/issues/15363.

Currently this package publishes .ts files in the package on npmjs, so that rules like noImplicitAny get picked up when you import { ... } from "api-ai-javascript".

The recommendation is to instead publish .js files with the corresponding .d.ts files. See the link at the top of this comment for the issue thread on the TypeScript repo about this.

An example of another repo that has made this change to their TS project publishing: https://github.com/Goyoo/node-k8s-client/pull/37.

drewsilcock avatar Aug 18 '17 09:08 drewsilcock

+1 for this.

I have strictNullChecks turned on which results in a bunch of compile errors from this library.

ghost avatar Dec 18 '17 21:12 ghost