rollup / rollup-plugin-commonjs

This module has moved and is now available at @rollup/plugin-commonjs / https://github.com/rollup/plugins/blob/master/packages/commonjs
MIT License
502 stars 126 forks source link

Include index.d.ts in npm package #382

Closed NotWoods closed 5 years ago

NotWoods commented 5 years ago

I added the index.d.ts file but forgot to include it in the "files" field in package.json, so its not distributed with the npm package.

@lukastaegert The typing file does not exists in the package available on npm registry. I think this part is missing in package.json file :

"files": [
    "dist/*.js",
    "dist/*.d.ts"
  ],

And maybe also :

 "typings": "index.d.ts",

Originally posted by @nicolashenry in https://github.com/rollup/rollup-plugin-commonjs/pull/363#issuecomment-479429969