-
Notifications
You must be signed in to change notification settings - Fork 60
readme: describe how to run tests #139
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
Conversation
526b4a7
to
5f206c7
Compare
5f206c7
to
7a2f1cc
Compare
510da5f
to
45def97
Compare
7a2f1cc
to
7f26ec4
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.
Hi! Thank you for the patch. Generally ok.
I suggest to returning to it after the #137 is merged, because it may affect this patch.
413bef4
to
bc75d65
Compare
7f26ec4
to
f158a3b
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.
Hi! Thank you for the patch.
Am I correct in assuming that we can't run all the tests at once now and have to go through the subpackages to run their tests?
bc75d65
to
cc58bf1
Compare
f158a3b
to
6240fa8
Compare
cc58bf1
to
b6f8631
Compare
Oh, that's a good question! For some reason I forget to check this possibility, but it seems like you can run them with I'll definitely make some improvements to #137 and rework this PR |
b6f8631
to
8314aa6
Compare
f5ef71d
to
922321c
Compare
@DifferentialOrange please fix conflicts |
Sorry, I forgot to rebase the branch after last change in #137 |
922321c
to
491e9aa
Compare
6240fa8
to
ad7fdf8
Compare
I commited and pushed all changes related to #137 updates |
ad7fdf8
to
e3ce790
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.
LGTM
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.
LGTM
aa80991
to
bd1263f
Compare
e3ce790
to
8d24a5b
Compare
bd1263f
to
861a33f
Compare
8d24a5b
to
e4c8da6
Compare
Describe how to run connector tests (with submodules) and test requirements. Closes #106
e4c8da6
to
4921aff
Compare
Prerequirements: merging #137 (there is no need to describe environment preparation if it is handled automatically).
Describe how to run connector tests (with submodules) and test requirements.
Closes #106