Exercise sync, part 1 - #1127
Conversation
|
This PR touches files which potentially affect the outcome of the tests of an exercise. This will cause all students' solutions to affected exercises to be re-tested. If this PR does not affect the result of the test (or, for example, adds an edge case that is not worth rerunning all tests for), please add the following to the merge-commit message which will stops student's tests from re-running. Please copy-paste to avoid typos. For more information, refer to the documentation. If you are unsure whether to add the message or not, please ping |
|
If you look at the old |
depial
left a comment
There was a problem hiding this comment.
LG2M!
The update you made for runtests.jl certainly makes a lot of sense for readability during troubleshooting, especially in the online editor.
|
I didn't prevent retesting (1026 completions for Bob): mainly because I forgot, but also these are big changes to the tests. |
Running
configlet sync -uidentified a few practice exercises that needed updating.I added a template for Bob, but ran into errors when regenerating the tests: one of the string literals confuses the Julia parser.
A few potential workarounds failed, so I've marked that test as
include = falseintests.toml.Conversely, a few older tests were excluded for reasons that no longer seem relevant, so I removed the
include = falselines.Finally, I added
.pycfiles (from thegenerate_tests.pyruns) to.gitignore.This seems enough complexity for one PR, so I'll template and update
connect,perfect-numbersandswift-schedulingseparately