-
Notifications
You must be signed in to change notification settings - Fork 8
Add CI for existing tests using github actions #52
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
Merged
Merged
Changes from 16 commits
Commits
Show all changes
17 commits
Select commit
Hold shift + click to select a range
c4260fb
add mac + linux test runners
dmorra12 1efb97a
fix env expansion
dmorra12 edcb6ff
fix env expansion again
dmorra12 1b6a055
just be explicit about args
dmorra12 909ea90
some tests weren't tests
dmorra12 756e760
upload logs on test failure
dmorra12 59943a9
only upload logs if a test fails
dmorra12 5f1d817
Merge remote-tracking branch 'origin/main' into add_actions
dmorra12 390519a
rm large test filter
dmorra12 be7763d
fix mac builds
dmorra12 5e7a34a
fix more cross-platform test issues
dmorra12 d804cc8
disable flaky test
dmorra12 ba6f587
test on arm linux too
dmorra12 7d2a1fe
fix os tag name
dmorra12 344d70a
Merge remote-tracking branch 'origin/main' into add_actions
dmorra12 6fbe990
fix merge
dmorra12 09f8138
comments from review
dmorra12 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,49 @@ | ||
| name: CI | ||
|
|
||
| on: [push, pull_request] | ||
|
|
||
| jobs: | ||
| test: | ||
| runs-on: ${{ matrix.os }} | ||
| strategy: | ||
| fail-fast: false | ||
| matrix: | ||
| include: | ||
| - os: ubuntu-latest | ||
| - os: ubuntu-24.04-arm | ||
| - os: macos-latest | ||
| bazel_flags: --config=apple_silicon | ||
|
|
||
| steps: | ||
| - name: Checkout code | ||
| uses: actions/checkout@v6 | ||
|
|
||
| - name: Install Bazel | ||
| uses: bazel-contrib/setup-bazel@0.18.0 | ||
|
dmorra12 marked this conversation as resolved.
|
||
| with: | ||
| # Avoid downloading Bazel every time. | ||
| bazelisk-cache: true | ||
| # Store build cache per workflow. | ||
| disk-cache: ${{ github.workflow }}-${{ matrix.os }} | ||
| # Share repository cache between workflows. | ||
| repository-cache: true | ||
|
|
||
| - name: Build all targets | ||
| run: | | ||
| bazel build //... \ | ||
| --verbose_failures \ | ||
| ${{ matrix.bazel_flags }} | ||
|
|
||
| - name: Run tests | ||
| run: | | ||
| bazel test //... \ | ||
| --verbose_failures \ | ||
| --test_output=errors \ | ||
| ${{ matrix.bazel_flags }} | ||
|
|
||
| - name: Upload Bazel test logs | ||
| uses: actions/upload-artifact@v7 | ||
| if: failure() # Upload artifacts only if a step failed | ||
| with: | ||
| name: bazel-test-logs | ||
|
dmorra12 marked this conversation as resolved.
Outdated
|
||
| path: bazel-testlogs | ||
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.