From 505243a633b56b607991cd4b7523450202a9d4f1 Mon Sep 17 00:00:00 2001 From: Antti Kervinen Date: Fri, 27 Sep 2024 15:58:26 +0300 Subject: [PATCH] ci: verify code generation and generated files in repo Signed-off-by: Antti Kervinen --- .github/workflows/ci.yml | 13 +++++++++++++ Makefile | 7 +++++++ 2 files changed, 20 insertions(+) diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml index 847b05e..dc069cb 100644 --- a/.github/workflows/ci.yml +++ b/.github/workflows/ci.yml @@ -50,9 +50,22 @@ jobs: run: | echo "${{ github.workspace }}/bin" >> $GITHUB_PATH + - name: Install protoc and plugins + run: | + sudo make install-protoc + make install-protoc-dependencies install-ttrpc-plugin + + - name: Force regeneration of protobuf files on build + run: | + touch pkg/api/api.proto + - run: | make all + - name: Validate files in git have not been changed + run: | + make validate-repo-changes-are-staged + linters: name: Linters runs-on: ${{ matrix.os }} diff --git a/Makefile b/Makefile index 9e2a87f..81f8638 100644 --- a/Makefile +++ b/Makefile @@ -166,6 +166,13 @@ vet: golangci-lint: $(Q)$(GOLANG_CILINT) run +validate-repo-changes-are-staged: + $(Q)test -z "$$(git status --short | tee /dev/stderr)" || { \ + echo "Repository has unstaged changes."; \ + echo "Please make sure to commit all changes, including generated files."; \ + exit 1; \ + } + # # proto generation targets #