Skip to content

Commit 0de8469

Browse files
fulldecentdrinckes
andauthored
Use Mac for testing edge cases differing across processors (#665)
* Buy a Mac Mini M1 * Update main.yml * remove clang check from macos test clang isn't installed by default --------- Co-authored-by: Doug Rinckes <[email protected]>
1 parent b4228c9 commit 0de8469

File tree

1 file changed

+13
-0
lines changed

1 file changed

+13
-0
lines changed

.github/workflows/main.yml

Lines changed: 13 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -14,6 +14,19 @@ jobs:
1414
- name: check formatting
1515
run: cd ${OLC_PATH} && bash clang_check.sh
1616

17+
# C implementation. Lives in c/, tested with bazel.
18+
# Use Mac with Apple Silicon for this test because of different floating point math
19+
# which can affect test results https://github.com/google/open-location-code/issues/652
20+
test-c-macos-latest:
21+
runs-on: macos-latest
22+
env:
23+
OLC_PATH: c
24+
steps:
25+
- uses: actions/checkout@v4
26+
- uses: bazelbuild/setup-bazelisk@v3
27+
- name: test
28+
run: bazel test --test_output=all ${OLC_PATH}:all
29+
1730
# C++ implementation. Lives in cpp/, tested with bazel.
1831
test-cpp:
1932
runs-on: ubuntu-latest

0 commit comments

Comments
 (0)