Skip to content

Commit

Permalink
build: Don't run commit lint on release and it checks the entire hist…
Browse files Browse the repository at this point in the history
…ory and old commits don't lint correctly
  • Loading branch information
tristanmenzel committed Oct 30, 2024
1 parent 818f1aa commit a8d41e2
Show file tree
Hide file tree
Showing 3 changed files with 10 additions and 1 deletion.
7 changes: 6 additions & 1 deletion .github/workflows/ci-all.yml
Original file line number Diff line number Diff line change
@@ -1,5 +1,10 @@
on:
workflow_call:
inputs:
run-commit-lint:
required: false
type: boolean
default: false

jobs:
ci-algo-ts:
Expand Down Expand Up @@ -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
Expand Down
2 changes: 2 additions & 0 deletions .github/workflows/pr.yml
Original file line number Diff line number Diff line change
Expand Up @@ -17,3 +17,5 @@ jobs:
ci-all:
name: 'Build and test all packages'
uses: ./.github/workflows/ci-all.yml
with:
run-commit-lint: true
2 changes: 2 additions & 0 deletions .github/workflows/release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,8 @@ jobs:
ci:
name: Build and test
uses: ./.github/workflows/ci-all.yml
with:
run-commit-lint: false

release:
name: Release
Expand Down

0 comments on commit a8d41e2

Please sign in to comment.