Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: add types export to exports object #247

Merged
merged 1 commit into from
Jun 21, 2022
Merged

fix: add types export to exports object #247

merged 1 commit into from
Jun 21, 2022

Conversation

SebDuf
Copy link
Contributor

@SebDuf SebDuf commented Jun 20, 2022

Add types export to exports field so TypeScript 4.7 is able to the the types when using the module resolution for Node 16+.

The types inside the exports object has to be first for TypeScript to be happy (see this issue: microsoft/TypeScript#46334).

@lbenie lbenie changed the title Add types export to exports object fix: add types export to exports object Jun 21, 2022
@lbenie
Copy link
Owner

lbenie commented Jun 21, 2022

Thanks for the contribution, could you amend the commit message so that it follows the conventional commits convention because it fails the linter

@lbenie
Copy link
Owner

lbenie commented Jun 21, 2022

@all-contributors please add @SebDuf for code

@allcontributors
Copy link
Contributor

@lbenie

I've put up a pull request to add @SebDuf! 🎉

@SebDuf
Copy link
Contributor Author

SebDuf commented Jun 21, 2022

@lbenie Done!

@lbenie lbenie merged commit 3fe8fbd into lbenie:main Jun 21, 2022
@lbenie
Copy link
Owner

lbenie commented Jun 21, 2022

🎉 This PR is included in version 1.1.33 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants