-
Notifications
You must be signed in to change notification settings - Fork 445
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
Set up Bazel build for Travis #137
Comments
I couldn't figure out how to run two separate things in travis, do we just do them sequentially? Once you've done this I volunteer to do the same thing for setup.py :) This will mean I don't have to manually test this stuff when I make releases, which should speed stuff up a lot. |
@davidzchen — are you working on this? I was about to file the same issue, good thing you beat me to it. Getting Bazel to work well with Travis CI is a bit of a project, but I've solved this issue previously on another project; see my Let me know if you have any questions or need some help, I'm happy to help add Bazel support to this repo as well. |
We have done for go-jsonnet already, but not here. It can be done similarly if it's worth it, |
The Travis build went away, but we now have a GitHub actions workflow, which does run the Bazel build: https://github.com/google/jsonnet/blob/master/.github/workflows/build_and_test.yml#L70-L80 |
…e#137) * For compatability, emit the \n within the library, not the commandline tool * Also add manifestString to library
Currently, we are only running make test in the Travis build.
The text was updated successfully, but these errors were encountered: