Skip to content

Conversation

@sxwei123
Copy link

Add type definitions to opts, opts.cmp and opts.cycles.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling aab1c68 on sxwei123:master into 49e230d on epoberezkin:master.

@epoberezkin
Copy link
Owner

Hey @sxwei123 - looks cool :)

Is this PR opened by mistake or you propose to merge your fork back in?

@epoberezkin
Copy link
Owner

Most likely you just continued committing to the same branch where the original PR was opened from - correct?...

@sxwei123
Copy link
Author

sxwei123 commented Jan 3, 2021

@epoberezkin
Oh sorry. I think I did something wrong.
I created a new stringify package with Typescript and es6 based on your code.
Please closed this PR.

@sxwei123
Copy link
Author

sxwei123 commented Jan 3, 2021

Create another PR to improve the comparator function typing:
#47

@sxwei123 sxwei123 closed this Jan 3, 2021
@epoberezkin
Copy link
Owner

No problem - I figured it was accidental - thanks for confirming!

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