-
-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
run doc example code as a test suite #52
Comments
This seems like kind of an annoying project. Are you really going to do this? It seems like there would be better uses of your time. |
It would be a good thing to have at some point. Not urgent though. Also, I suspect, not all that hard to do. |
And maintaining the wiki when we change the language is kind of annoying too. This might be less work in the long run. |
Actually, a better way would be to have some annotations in the wiki, which can be extracted as code, run automatically within julia, and the real output used. Given the wiki is actually just a git repo makes this quite easy to pull off. |
I thought about that, but I think using it as a test suite is for the best. If the output changes, you really want to actually examine it and make sure that the surrounding text still makes sense and applies. |
Doesn't seem like this can/should block 1.0. Reprioritize? |
No, it shouldn't. Running through them all by hand should be fine. Can automate later. |
This calls for champagne. 🍷 |
+1 |
* `copy!()` -> `copyto!()` * Quash more 0.7 compatibility bugs * Require at least Compat 0.38
…ngle Add exercise: pascals-triangle
Check output type of `prepare_call` more carefully. Fixes #51.
The examples in the wiki should all be exactly what the interpreter actually produces. We should automate comparing the output of the interpreter with what is in the wiki, thereby ensuring accuracy of all the examples and serving as an additional set of integration tests. One issue is when to clear variables and state. An obvious choice is to retain state within sections (or subsections, or whatever).
The text was updated successfully, but these errors were encountered: