Skip to content

Commit ea3578d

Browse files
committed
Update workflow as per review (BLD-6874)
1 parent 4e7a668 commit ea3578d

File tree

1 file changed

+9
-11
lines changed

1 file changed

+9
-11
lines changed

.github/workflows/build-conquest-python.yml

+9-11
Original file line numberDiff line numberDiff line change
@@ -4,21 +4,19 @@ name: Build Conquest Python
44
on: # yamllint disable-line rule:truthy
55
workflow_dispatch:
66
inputs:
7-
deploy:
8-
type: choice
9-
description: 'Whether to upload conquest python build to artifactory'
10-
options:
11-
- 'no-upload'
12-
- 'upload'
7+
artifactory-push:
8+
description: "Push result to Artifactory"
9+
default: false
10+
type: boolean
1311
platforms:
1412
description: 'Platform to build for, eg. '
1513
options:
1614
- 'rocky8'
17-
- 'ubuntu22'
15+
- 'ubuntu'
1816
- 'macos'
1917
- 'windows'
2018
push:
21-
files:
19+
paths:
2220
- '.github/workflows/build-conquest-python.yml'
2321
- 'build_conquest_python.py'
2422
- 'ccdc/thirdparty/package.py'
@@ -42,7 +40,7 @@ jobs:
4240
echo runson=ubuntu-latest >> $GITHUB_OUTPUT
4341
echo container=ccdcrepository.azurecr.io/conan/rocky8-gcc10:latest >> $GITHUB_OUTPUT
4442
echo python=python3 >> $GITHUB_OUTPUT
45-
elif [[ "${{ inputs.platforms }}" == "ubuntu22" ]]; then
43+
elif [[ "${{ inputs.platforms }}" == "ubuntu" ]]; then
4644
echo runson=ubuntu-latest >> $GITHUB_OUTPUT
4745
echo python=python3 >> $GITHUB_OUTPUT
4846
elif [[ "${{ inputs.platforms }}" == "macos" ]]; then
@@ -92,13 +90,13 @@ jobs:
9290
echo "archive_path=$archive_path" >> $GITHUB_ENV
9391
9492
- name: Set up JFrog CLI
95-
# if: ${{ inputs.deploy == 'upload' }}
93+
if: ${{ inputs.artifactory-push }}
9694
uses: jfrog/setup-jfrog-cli@v4
9795
env:
9896
JF_ENV_1: ${{ secrets.ARTIFACTORY_GH_CCDC_3RDPARTY_PYTHON_INTERPRETERS_READ_WRITE_EXPORT }}
9997

10098
- name: Upload conquest python to Artifactory
101-
# if: ${{ inputs.deploy == 'upload' }}
99+
if: ${{ inputs.artifactory-push }}
102100
shell: bash
103101
run: |
104102
# Upload the conquest python to Artifactory

0 commit comments

Comments
 (0)