Skip to content
This repository has been archived by the owner on Aug 29, 2023. It is now read-only.

Add Typescript support #145

Merged
merged 8 commits into from
Jun 10, 2021
Merged

Conversation

nazarhussain
Copy link
Contributor

With reference to the issue libp2p/js-libp2p#659 this PR will add typescript support to libp2p-tcp.

Copy link
Member

@vasco-santos vasco-santos left a comment

Choose a reason for hiding this comment

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

Thanks for looking into this @nazarhussain

I did a first pass review

package.json Outdated Show resolved Hide resolved
types/abortable-iterator/index.d.ts Outdated Show resolved Hide resolved
src/index.js Show resolved Hide resolved
src/listener.js Show resolved Hide resolved
src/typedefs.js Outdated Show resolved Hide resolved
@nazarhussain
Copy link
Contributor Author

@vasco-santos Hope changes looks fine now. I also wanna know why are not using the typescript itself? and using JSDoc to TS approach?

@lidel lidel requested a review from yusefnapora June 7, 2021 14:35
@nazarhussain
Copy link
Contributor Author

If changes in this PR looks good then I will create few more for other packages in libp2p which yet not supporting TS. Waiting for this PR to get approved.

Copy link

@yusefnapora yusefnapora left a comment

Choose a reason for hiding this comment

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

This looks really solid to me, and as far as I can tell all of @vasco-santos's prior feedback has been addressed. Thanks @nazarhussain 😄

@vasco-santos
Copy link
Member

Thanks @nazarhussain and @yusefnapora ❤️

@nazarhussain if you would like to add support in other repos, it will be super helpful. Please let us know in libp2p/js-libp2p#659 when you start working on any repo to avoid work collisions

@vasco-santos
Copy link
Member

Release 0.16.0 with this

@nazarhussain nazarhussain deleted the nh-ts-support branch June 10, 2021 13:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants