-
Notifications
You must be signed in to change notification settings - Fork 17
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
[WIP] Add tests everywhere #59
base: master
Are you sure you want to change the base?
Conversation
@vmarkovtsev tests failing because the version of sourced-ml package seems to be too outdated, compared to the git repo. We might wanna release a new one ? Else we can install it using git |
Wrote tests for as much stuff as possible. Currently |
@vmarkovtsev let's merge this after #61 so I can modify/add tests for new ml and apollo features |
Added Konst's changes from PR #66 so tests would not fail, will update the rest of the code asap |
#66 was merged |
Great, will rebase and take care of conflicts asap. Ill also update the doc regarding the preprocess command |
Signed-off-by: Romain Keramitas <[email protected]>
Signed-off-by: Romain Keramitas <[email protected]>
Signed-off-by: Romain Keramitas <[email protected]>
#58