-
Notifications
You must be signed in to change notification settings - Fork 2k
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Write tests + documentation to explain how to use this package with restify #125
Comments
That's great news, thanks! Looking forward to the documentation and tests :) |
I'm interested in trying this out as well... Could you post what steps you've taken? I don't mind doing a PR for doc updates |
@joelgriffith I couldn't work on that, but I can totally guide you in your contribution. The first thing to do, I think is to start with the tests: take the express tests and adapt them to use restify in there instead. Commit, push and create the pr early so I can help you along the way! 馃槈 Thank you! |
Followup to #85.
Apollo Server works out-of-the-box with Restify 馃帀 .
We still need to ensure the compatibility with it: we need to ensure it won't break anytime soon so we need to provide both tests and documentation for this!
The text was updated successfully, but these errors were encountered: