-
Notifications
You must be signed in to change notification settings - Fork 5
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
Upgrade resty dependency #8
Conversation
Update imports Update logger interface Fix tests
Awesome work ❤️ . |
@mauricioabreu why change the logger interface? It wouldn't be enough to keep the |
@lucasrodcosta I had to change the interface because resty got upgraded, and the new interface is different from v1 One thing I have noticed is that they created a interface for this. So we can replace our custom interface by |
I've changed the custom interface by |
Hey @mauricioabreu . Could you update the examples section in README? After this PR, a logger struct is needed to initialize |
Good gatch @KalilCazes Done! |
Update imports
Update logger interface
Fix tests