Skip to content

Commit

Permalink
Update clang-tidy action to stop breaking (#7808)
Browse files Browse the repository at this point in the history
* Switch clang-tidy action from macos-13 to macos-latest

`macos-latest` is actually macos-12 (macos13 is considered "beta" on the GHA runners). Hopefully this will fix the recent install snafus that are breaking clang-tidy.

* Bogus change to trigger check

* Update presubmit.yml

* Update presubmit.yml

* Update presubmit.yml

* Revert "Bogus change to trigger check"

This reverts commit a70f9ed.

* Update presubmit.yml
  • Loading branch information
steven-johnson authored Aug 28, 2023
1 parent 8ac1e1c commit 79d2be3
Showing 1 changed file with 10 additions and 2 deletions.
12 changes: 10 additions & 2 deletions .github/workflows/presubmit.yml
Original file line number Diff line number Diff line change
Expand Up @@ -28,13 +28,21 @@ jobs:
# a bit of a sluggard
check_clang_tidy:
name: Check clang-tidy
runs-on: macos-13
runs-on: macos-latest
steps:
- uses: actions/checkout@v3
# Workaround for https://github.com/actions/setup-python/issues/577
- name: Clean up python binaries
run: |
rm -f /usr/local/bin/2to3*;
rm -f /usr/local/bin/idle3*;
rm -f /usr/local/bin/pydoc3*;
rm -f /usr/local/bin/python3*;
rm -f /usr/local/bin/python3*-config;
- name: Install clang-tidy
run: |
brew update
brew install llvm@16 ninja coreutils
brew install llvm@16 ninja coreutils flatbuffers
- name: Run clang-tidy
run: |
export CC=/usr/local/opt/llvm/bin/clang
Expand Down

0 comments on commit 79d2be3

Please sign in to comment.