- 
                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 📦🚀  | 
    
| "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