-
Notifications
You must be signed in to change notification settings - Fork 41
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
Documentation source code can be tested #43
Comments
Implementation:
|
For the samples that need additional dependencies, do you have another issue for tracking that? |
@big-guy Can you elaborate? I don't quite understand what you mean. |
@big-guy There's more to that test code in these samples. I think they were built with an older version of Play and are not completely incompatible: changed APIs, different imports, missing dependencies. I spent about an hour on it but couldn't make it work. I think somebody with proper knowledge of the Play API would get the solution much faster. I created #74 with my findings. |
This good enough for now |
The text was updated successfully, but these errors were encountered: