Skip to content

Commit ad7e67b

Browse files
committed
Remove build-directory input in manual workflows
Workflow dispatch only allows upto 10 inputs, so we can fix build-directory to C:\build and remove it as input
1 parent d981c61 commit ad7e67b

File tree

2 files changed

+2
-8
lines changed

2 files changed

+2
-8
lines changed

.github/workflows/extension.yml

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -46,9 +46,6 @@ on:
4646
description: 'Test opcache mode'
4747
required: false
4848
default: 'on'
49-
build-directory:
50-
description: 'Directory to build the extension in'
51-
required: false
5249
jobs:
5350
get-extension-matrix:
5451
runs-on: ubuntu-latest
@@ -86,10 +83,10 @@ jobs:
8683
ts: ${{ matrix.ts }}
8784
args: ${{ inputs.args }}
8885
libs: ${{ inputs.libs }}
89-
build-directory: ${{ inputs.build-directory }}
9086
run-tests: ${{ inputs.run-tests }}
9187
test-runner: ${{ inputs.test-runner }}
9288
test-opcache-mode: ${{ inputs.test-opcache-mode }}
89+
build-directory: C:\build
9390

9491
artifacts:
9592
runs-on: ubuntu-latest

.github/workflows/pecl.yml

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -40,9 +40,6 @@ on:
4040
description: 'Test runner to use'
4141
required: false
4242
default: 'run-tests.php'
43-
build-directory:
44-
description: 'Directory to build the extension in'
45-
required: false
4643
jobs:
4744
get-extension-matrix:
4845
runs-on: ubuntu-latest
@@ -80,9 +77,9 @@ jobs:
8077
ts: ${{ matrix.ts }}
8178
args: ${{ inputs.args }}
8279
libs: ${{ inputs.libs }}
83-
build-directory: ${{ inputs.build-directory }}
8480
run-tests: ${{ inputs.run-tests }}
8581
test-runner: ${{ inputs.test-runner }}
82+
build-directory: C:\build
8683
env:
8784
artifact-naming-scheme: pecl
8885

0 commit comments

Comments
 (0)