-
Notifications
You must be signed in to change notification settings - Fork 18
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
Minor fixes #137
Minor fixes #137
Conversation
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.
Conditional approval, there is a test that is failing.
Base and Client files had some minor issues in the comments that have been fixed in this patch. Signed-off-by: Iker Pedrosa <ipedrosa@redhat.com>
091a374
to
affe392
Compare
Signed-off-by: Iker Pedrosa <ipedrosa@redhat.com>
affe392
to
2ea30be
Compare
I'm not really sure what is happening here, but the CI errors are the ones fixed at #138 and I already rebased my branch on top of it 😕 |
Hmm. Let's merge it and we'll see. |
This fixes it #143 |
I have know idea, I was mainly looking at CI result as well in this case. You can try to checkout the code and build it locally. |
While implementing shadow's test framework I discovered several issues that I've fixed