Don't run build workflow on draft PRs. (#20443)
This commit is contained in:
parent
59ff9c2eea
commit
120830681e
|
@ -4,6 +4,11 @@ name: build
|
|||
on:
|
||||
workflow_dispatch:
|
||||
pull_request:
|
||||
# The default types for pull_request are [ opened, synchronize, reopened ].
|
||||
# This is insufficient for our needs, since we're skipping stuff on PRs in
|
||||
# draft mode. By adding the ready_for_review type, when a draft pr is marked
|
||||
# ready, we run everything, including the stuff we'd have skipped up until now.
|
||||
types: [ opened, synchronize, reopened, ready_for_review ]
|
||||
push:
|
||||
branches:
|
||||
- main
|
||||
|
@ -12,6 +17,7 @@ on:
|
|||
jobs:
|
||||
# verify-changes determines if the changes are only for docs (website)
|
||||
verify-changes:
|
||||
if: github.event.pull_request.draft == false
|
||||
runs-on: ubuntu-latest
|
||||
outputs:
|
||||
is_docs_change: ${{ steps.get-changeddir.outputs.is_docs_change }}
|
||||
|
|
|
@ -1,6 +1,11 @@
|
|||
name: CI
|
||||
on:
|
||||
pull_request:
|
||||
# The default types for pull_request are [ opened, synchronize, reopened ].
|
||||
# This is insufficient for our needs, since we're skipping stuff on PRs in
|
||||
# draft mode. By adding the ready_for_review type, when a draft pr is marked
|
||||
# ready, we run everything, including the stuff we'd have skipped up until now.
|
||||
types: [ opened, synchronize, reopened, ready_for_review ]
|
||||
push:
|
||||
branches:
|
||||
- main
|
||||
|
@ -141,6 +146,7 @@ jobs:
|
|||
- setup-go-cache
|
||||
# Don't run this job for PR branches starting with 'ui/', 'backport/ui/', 'docs/', or 'backport/docs/'
|
||||
if: |
|
||||
github.event.pull_request.draft == false &&
|
||||
!startsWith(github.head_ref, 'ui/') &&
|
||||
!startsWith(github.head_ref, 'backport/ui/') &&
|
||||
!startsWith(github.head_ref, 'docs/') &&
|
||||
|
@ -162,6 +168,7 @@ jobs:
|
|||
name: Run Go tests with FIPS configuration
|
||||
# Only run this job for the enterprise repo if the PR branch doesn't start with 'ui/', 'backport/ui/', 'docs/', or 'backport/docs/'
|
||||
if: |
|
||||
github.event.pull_request.draft == false &&
|
||||
needs.setup.outputs.enterprise == 1 &&
|
||||
!startsWith(github.head_ref, 'ui/') &&
|
||||
!startsWith(github.head_ref, 'backport/ui/') &&
|
||||
|
@ -293,4 +300,4 @@ jobs:
|
|||
runs-on: ${{ fromJSON(needs.setup.outputs.compute-tiny) }}
|
||||
steps:
|
||||
- run: |
|
||||
tr -d '\n' <<< '${{ toJSON(needs.*.result) }}' | grep -q -v -E '(failure|cancelled)'
|
||||
tr -d '\n' <<< '${{ toJSON(needs.*.result) }}' | grep -q -v -E '(failure|cancelled)'
|
||||
|
|
Loading…
Reference in New Issue