-
Notifications
You must be signed in to change notification settings - Fork 397
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
feat: add support for Restify #118
Comments
Im not against it if you will maintain it |
My resources are very limited at the moment (I think it goes for most of us here), but I definitely would like support for Restify, so I will try to implement it in the future, if no one does this before me of course. |
Can anyone point to the areas that need adaption ? We are interested in using restify too. |
I've added support for custom drivers, so all you need to do is extend the Once you will be satisfied with the restify driver, you can create a PR with the new driver, but you also have to change test suite for the new driver 😉 |
Stale issue message |
We are moving away from custom/multiple driver support. RC will be shipped with a fixed driver. |
Restify has a very similar API to Express, but nevertheless has issues when used with Express driver. Probably, it will need it's own driver for clarity and maintainability.
The text was updated successfully, but these errors were encountered: