Skip to content
This repository has been archived by the owner on Feb 22, 2019. It is now read-only.

Remove async dependency #66

Merged
merged 1 commit into from
Jul 18, 2018
Merged

Remove async dependency #66

merged 1 commit into from
Jul 18, 2018

Conversation

blowery
Copy link
Contributor

@blowery blowery commented Jul 17, 2018

It does not appear we use it anywhere.

@blowery
Remove async dependency
d9e2981 
It does not appear we use it anywhere.
@blowery blowery self-assigned this Jul 17, 2018
@blowery blowery requested review from sirreal and Tug July 18, 2018 00:43
sirreal
sirreal approved these changes Jul 18, 2018
View reviewed changes
Copy link
Member

@sirreal sirreal left a comment

Choose a reason for hiding this comment

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

I did a quick code search, didn't see it in use. Tests pass. Nice spot!

🚢

@blowery blowery merged commit 6dd21c0 into master Jul 18, 2018
@blowery blowery deleted the remove/async-dependency branch July 18, 2018 13:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@sirreal sirreal sirreal approved these changes

@Tug Tug Awaiting requested review from Tug

Assignees

@blowery blowery

Labels
needs review
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@blowery @sirreal