-
Notifications
You must be signed in to change notification settings - Fork 22
feat: Add Typescript types #72
feat: Add Typescript types #72
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for this!
🎉 This PR is included in version 5.1.0 🎉 The release is available on:
Your semantic-release bot 📦🚀 |
@@ -60,6 +62,7 @@ | |||
"files": [ | |||
"*.js" | |||
], | |||
"types": "index.d.ts", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
index.d.ts
is not included in files
, so it's not part of the npm package:
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
D'oh! I'll get that fixed and a new release out right away.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixed in 5.1.1
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I could have sworn the types field was included by default. Main is, but types aren't I guess :(
https://docs.npmjs.com/files/package.json
Thanks @OliverJAsh and @chrisbreiding!
* feat: typings * fix: revert cypress version change * make eslint plugin happy, but also don't lint types file * chore: update yarn.lock Co-authored-by: Chris Breiding <[email protected]>
Fixes #67