Merge pull request #113 from askervin/5XD_ci_check_generated

ci: verify that code generation works and results match

.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-no-changes
+
   linters:
     name: Linters
     runs-on: ${{ matrix.os }}

Makefile

@@ -166,6 +166,13 @@ vet:
 golangci-lint:
 	$(Q)$(GOLANG_CILINT) run
 
+validate-repo-no-changes:
+	$(Q)test -z "$$(git status --short | tee /dev/stderr)" || { \
+		echo "Repository has changes."; \
+		echo "Please make sure to commit all changes, including generated files."; \
+		exit 1; \
+	}
+
 #
 # proto generation targets
 #