-
Notifications
You must be signed in to change notification settings - Fork 0
More test coverage #4
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
b210def
to
5e62c52
Compare
5e62c52
to
5a9b8d0
Compare
Why don't we run all those tests during CI builds? |
@cv711 I had issues with downloading and building capnp in circle ci. One have to build it from source and a special version for node 8. And then install the npm capnp lib. |
@karlnorling we could run the tests inside the sandbox container we are building no? |
e1c49ca
to
c05d168
Compare
c05d168
to
aaace34
Compare
@cv711 this is a lot more work than I thought - and I'm not sure if updating the
Right now it's failing on
And I have no idea how to install futex kernel tool on alpine distro. |
b1fef28
to
c4facc2
Compare
c4facc2
to
4183ad1
Compare
401f557
to
0f5d860
Compare
0f5d860
to
71124d5
Compare
787bbad
to
15ca37a
Compare
15ca37a
to
65706ac
Compare
Adding some more tests.