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

Make wordnet APIs async #3

Merged
merged 6 commits into from
Jun 10, 2020
Merged

Make wordnet APIs async #3

merged 6 commits into from
Jun 10, 2020

Conversation

hijonathan
Copy link
Collaborator

Hi there 👋 .

Admittedly I got a little carried away with this, but I was hoping to use this library and thought it could use some improvements, namely updating it to use promises.

I've included tests and (loosely) verified that it passes with the existing codebase, though the API change is breaking. I made a branch for just tests and can merge that first if you'd like. I also used tape since that seems to be what you use in other repos.

Sorry for the big PR. I tried to make as few changes as possible, but since everything seems to use callbacks it was unavoidable.

@hijonathan hijonathan changed the title Make wordnet APIs a little easier to work with Make wordnet APIs async Jun 9, 2020
@dariuszdziuk
Copy link
Collaborator

Hi @hijonathan, glad that you found the code useful and chose to spend time on modernising it!

@dariuszdziuk dariuszdziuk merged commit 49c1493 into words:master Jun 10, 2020
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.

2 participants