From a8d41e28000f1c566e79916f202b9aef90beb0c5 Mon Sep 17 00:00:00 2001 From: Tristan Menzel Date: Tue, 29 Oct 2024 18:08:36 -0700 Subject: [PATCH] build: Don't run commit lint on release and it checks the entire history and old commits don't lint correctly --- .github/workflows/ci-all.yml | 7 ++++++- .github/workflows/pr.yml | 2 ++ .github/workflows/release.yml | 2 ++ 3 files changed, 10 insertions(+), 1 deletion(-) diff --git a/.github/workflows/ci-all.yml b/.github/workflows/ci-all.yml index e072e1fc..6b4de180 100644 --- a/.github/workflows/ci-all.yml +++ b/.github/workflows/ci-all.yml @@ -1,5 +1,10 @@ on: workflow_call: + inputs: + run-commit-lint: + required: false + type: boolean + default: false jobs: ci-algo-ts: @@ -35,7 +40,7 @@ jobs: pre-run-script: mv packages-temp/algo-ts packages/algo-ts/dist node-version: 20.x run-build: true - run-commit-lint: true + run-commit-lint: ${{ inputs.run-commit-lint }} build-script: npm run build audit-script: npm run audit upload-artifact-name: puya-ts diff --git a/.github/workflows/pr.yml b/.github/workflows/pr.yml index e2d280ff..13264b52 100644 --- a/.github/workflows/pr.yml +++ b/.github/workflows/pr.yml @@ -17,3 +17,5 @@ jobs: ci-all: name: 'Build and test all packages' uses: ./.github/workflows/ci-all.yml + with: + run-commit-lint: true diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml index 570ae623..f7dd5ff0 100644 --- a/.github/workflows/release.yml +++ b/.github/workflows/release.yml @@ -19,6 +19,8 @@ jobs: ci: name: Build and test uses: ./.github/workflows/ci-all.yml + with: + run-commit-lint: false release: name: Release