Closed varunagrawal closed 2 years ago
Why is this on your A->B right now?
It isn't. It was some pending changes I wanted to get out of the way before I lost them to the void.
CI fails
Yup, my PR comment explicitly asks Yetong to look into this since he's the owner of this codebase.
The issue is that the Python tests for the Jumping Robot were never added to the CI until now, that's why you're seeing the tests fail in this PR.
Sorry I just noticed this PR review. It seems the two libraries matplotlib
and yaml
may not be installed in the travis Python version. I'll make a commit into this branch to handle that.
The previous CI fail due to the pyyaml
package dependency, and the update on the Link constructor. After revising the issues, all checks have passed.
Adds tests for the Jumping Robot to the CI.
Currently 2 tests fail. @yetongumich can you please also look into this since I will not get to it until the 11/30.