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

changed two function for one #13

Closed
wants to merge 4 commits into from
Closed

changed two function for one #13

wants to merge 4 commits into from

Conversation

renanbastos93
Copy link

Hey dude,

I propose this change, because your code did unreadable. So, now it is one function that return sync or for callback. That is, will to be more performance with less alocation memory.

see ya.

@renanbastos93
Copy link
Author

renanbastos93 commented Mar 14, 2018

@michaelrhodes I don't know execute this lib zuul.

Why you use the zuul, if deployed is fail?

image

@feross
Copy link
Collaborator

feross commented Mar 15, 2018

No thanks. The current API is fine.

@feross feross closed this Mar 15, 2018
@renanbastos93
Copy link
Author

Oh, okay.

I'm sorry, I wait help you other moment.

see ya

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.

None yet

2 participants