-
Notifications
You must be signed in to change notification settings - Fork 8
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
[TEST] tidy up tests and add all method combination tests in a new file #73
[TEST] tidy up tests and add all method combination tests in a new file #73
Conversation
…te file Signed-off-by: Lydia Buntrock <lydia.buntrock@fu-berlin.de>
a2392f0
to
c210808
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some for loops are broken.
Thanks for the good suggestions for improvement! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice work!
Signed-off-by: Lydia Buntrock <lydia.buntrock@fu-berlin.de>
36cd55f
to
760cf59
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a few small things! Nice work~
Signed-off-by: Lydia Buntrock <lydia.buntrock@fu-berlin.de>
Resolves #69