Skip to content
This repository was archived by the owner on Aug 4, 2021. It is now read-only.

Include index.d.ts in npm package#382

Merged
lukastaegert merged 1 commit intorollup:masterfrom
NotWoods:patch-1
Apr 4, 2019
Merged

Include index.d.ts in npm package#382
lukastaegert merged 1 commit intorollup:masterfrom
NotWoods:patch-1

Conversation

@NotWoods
Copy link
Member

@NotWoods NotWoods commented Apr 3, 2019

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 #363 (comment)

Copy link
Member

@lukastaegert lukastaegert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks a lot!

@lukastaegert lukastaegert merged commit f727dc9 into rollup:master Apr 4, 2019
@NotWoods NotWoods deleted the patch-1 branch April 4, 2019 04:49
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants