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

Feature: TS declaration support #25

Merged
merged 3 commits into from
Dec 23, 2021

Conversation

kamikazebr
Copy link
Member

@kamikazebr kamikazebr commented Dec 18, 2021

It include automation to autogenerated typescript definitions and bundled on one single file to distribute together with the package.

Running yarn build will generate the type definition on dist/types/index.d.ts

@kamikazebr kamikazebr changed the title Feat/ts declaration support Feature: TS declaration support Dec 18, 2021
package.json Outdated Show resolved Hide resolved
@@ -98,9 +105,9 @@
"react-blockies": "^1.4.0",
"react-display-name": "^0.2.3",
"react-onclickout": "^2.0.8",
"react-spring": "^7.2.11",
"react-spring": "7.2.10",
Copy link
Member Author

Choose a reason for hiding this comment

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

The version 7.2.11 not exist anymore, so was neeed rollback to resolve some conflict on types definition.

"recursive-copy": "^2.0.9",
"use-inside": "^0.2.0"
"use-inside": "https://github.com/aragon/use-inside#16f321e499d8f89e9b0fb0999e2d21db2b119bbf"
Copy link
Member Author

Choose a reason for hiding this comment

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

On bundle api-extractor cannot found definition of use-inside on the last version published on npm, but on currenty master (referenced by commit for safety install), have the TS definitions updated

Copy link

@tmoutinho tmoutinho left a comment

Choose a reason for hiding this comment

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

Tested and works great.

@0xGabi 0xGabi merged commit 959c02c into 1Hive:master Dec 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants