Skip to content

Commit cc5cf76

Browse files
committed
fix up yml file
1 parent c3f0211 commit cc5cf76

File tree

1 file changed

+6
-9
lines changed

1 file changed

+6
-9
lines changed

.github/workflows/wheels.yml

Lines changed: 6 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,4 @@
11
name: Build Wheels for Python SDK
2-
32
on:
43
# Allow manual triggering of the workflow
54
workflow_dispatch:
@@ -15,13 +14,11 @@ jobs:
1514
runs-on: ${{ matrix.os }}
1615
strategy:
1716
matrix:
18-
# Macos 13 is an intel runner and macos 14 is an apple silicon runner
17+
# macOS 13 is an Intel runner and macOS 14 is an Apple Silicon runner
1918
os: [ubuntu-22.04, ubuntu-22.04-arm, windows-latest, macos-13, macos-14]
20-
21-
2219
steps:
2320
- uses: actions/checkout@v4
24-
21+
2522
# Need to grab the SDK version for the wheel name
2623
- name: Extract SDK Version
2724
run: echo "SDK_VERSION=$(python -c 'import version; print(version.SDK_VERSION)')" >> $GITHUB_ENV
@@ -36,11 +33,11 @@ jobs:
3633
CIBW_TEST_COMMAND: "python -m pytest {project}/src/onepassword/test_client.py"
3734
MACOSX_DEPLOYMENT_TARGET: 11.0 # Required for building on macOS
3835
OP_SERVICE_ACCOUNT_TOKEN: ${{ secrets.TEST_SERVICE_ACCOUNT_TOKEN }}
39-
CIBW_ENVIRONMENT_PASS_LINUX: OP_SERVICE_ACCOUNT_TOKEN # to pass in the SA token, for some reason linux doesn't read the env variables
36+
CIBW_ENVIRONMENT_PASS_LINUX: OP_SERVICE_ACCOUNT_TOKEN # to pass in the SA token, for some reason Linux doesn't read the env variables correctly
4037

4138
uses: pypa/[email protected]
42-
39+
4340
- uses: actions/upload-artifact@v4
4441
with:
45-
name: onepassword-sdk-${{ env.SDK_VERSION }}-${{ matrix.os }}-${{ strategy.job-index }}
46-
path: ./wheelhouse/*.whl
42+
name: onepassword-sdk-${{ env.SDK_VERSION }}-${{ matrix.os }}
43+
path: ./wheelhouse/*.whl

0 commit comments

Comments
 (0)