Skip to content

Commit

Permalink
GHA: build: simplify action (#271)
Browse files Browse the repository at this point in the history
No need to setup protoc twice. `make build` also invokes `deps`.

Signed-off-by: Isabella do Amaral <[email protected]>
  • Loading branch information
isinyaaa authored Aug 19, 2024
1 parent 721cdd2 commit 22c93da
Showing 1 changed file with 12 additions and 21 deletions.
33 changes: 12 additions & 21 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,18 +2,18 @@ name: Build
on:
push:
branches:
- 'main'
- "main"
pull_request:
paths-ignore:
- 'LICENSE*'
- 'DOCKERFILE*'
- '**.gitignore'
- '**.md'
- '**.txt'
- '.github/ISSUE_TEMPLATE/**'
- '.github/dependabot.yml'
- 'docs/**'
- 'scripts/**'
- "LICENSE*"
- "DOCKERFILE*"
- "**.gitignore"
- "**.md"
- "**.txt"
- ".github/ISSUE_TEMPLATE/**"
- ".github/dependabot.yml"
- "docs/**"
- "scripts/**"
jobs:
build:
runs-on: ubuntu-latest
Expand All @@ -22,22 +22,13 @@ jobs:
- name: Setup Go
uses: actions/setup-go@v5
with:
go-version: '1.21'
- name: Install Protoc
uses: arduino/setup-protoc@v3
with:
version: "24.3"
repo-token: ${{ secrets.GITHUB_TOKEN }}
go-version: "1.21"
- name: Set up Python
uses: actions/setup-python@v5
with:
python-version: 3.9
- name: Install makefile dependencies
run: make deps
- name: Clean
run: make clean
- name: Build
run: make build
run: make clean build
- name: Check if there are uncommitted file changes
run: |
clean=$(git status --porcelain)
Expand Down

0 comments on commit 22c93da

Please sign in to comment.