-
Notifications
You must be signed in to change notification settings - Fork 259
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
Add GitHub Actions. #8
Conversation
Apparently, actions won't work until we make the repo public. |
9b83f5d
to
49225ea
Compare
I'm not going to merge this until I can actually verify it works, so I guess this PR will sit open for a week. |
Note to self: Add a check that bans fixup commits. |
53f14b5
to
f3b742f
Compare
OMG we are going to need to do something about these slow builds. |
How about using the builtin bazel cache with a R2 bucket? Happy to do a PR/add on top if so. It would be problematic for forked repos but I guess the step that enables it can be conditional to what the repo is. |
.github/workflows/test.yml
Outdated
sudo apt-get install -y build-essential git clang libc++-dev | ||
- name: bazel tests | ||
run: | | ||
bazel test --verbose_failures --test_output=errors //... |
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.
--show_timestamps is very helpful. I also suggest -k
.github/workflows/test.yml
Outdated
sudo apt-get install -y build-essential git clang libc++-dev | ||
- name: bazel tests | ||
run: | | ||
bazel test --verbose_failures --test_output=errors //... |
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.
We run @capnp-cpp//... //... -- -@capnp-cpp//src/capnp:fuzz-test
, do we want capnp tests here too?
f3b742f
to
9ddee5c
Compare
No description provided.