Skip to content

Commit 86f53e7

Browse files
committed
Use absolute paths for all upload-artifact-verified calls
Convert all path arguments to upload-artifact-verified action from relative paths to absolute paths using ${{ github.workspace }}. This eliminates any ambiguity in path resolution and follows the KISS principle - explicit, no magic. Files updated: - main.yml - wstest.yml - wheels.yml - wheels-docker.yml - wheels-arm64.yml Note: This work was completed with AI assistance (Claude Code).
1 parent f3163c5 commit 86f53e7

File tree

5 files changed

+19
-20
lines changed

5 files changed

+19
-20
lines changed

.github/workflows/main.yml

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -97,7 +97,7 @@ jobs:
9797
uses: wamp-proto/wamp-cicd/actions/upload-artifact-verified@main
9898
with:
9999
name: coverage-report-combined-with-nvx
100-
path: docs/_build/html/coverage-combined-with-nvx/
100+
path: ${{ github.workspace }}/docs/_build/html/coverage-combined-with-nvx/
101101
retention-days: 14
102102

103103
- name: Code test coverage (Combined Twisted + asyncio) - WITHOUT NVX
@@ -108,7 +108,7 @@ jobs:
108108
uses: wamp-proto/wamp-cicd/actions/upload-artifact-verified@main
109109
with:
110110
name: coverage-report-combined-without-nvx
111-
path: docs/_build/html/coverage-combined-without-nvx/
111+
path: ${{ github.workspace }}/docs/_build/html/coverage-combined-without-nvx/
112112
retention-days: 14
113113

114114
- name: Unit test - Twisted / trial - WITH NVX
@@ -208,7 +208,7 @@ jobs:
208208
uses: wamp-proto/wamp-cicd/actions/upload-artifact-verified@main
209209
with:
210210
name: serdes-test-results-${{ matrix.python-env }}
211-
path: test-results/serdes-${{ matrix.python-env }}/
211+
path: ${{ github.workspace }}/test-results/serdes-${{ matrix.python-env }}/
212212
retention-days: 7
213213

214214
documentation:
@@ -271,7 +271,7 @@ jobs:
271271
uses: wamp-proto/wamp-cicd/actions/upload-artifact-verified@main
272272
with:
273273
name: documentation
274-
path: docs/_build/html/
274+
path: ${{ github.workspace }}/docs/_build/html/
275275
retention-days: 14
276276

277277
# FIXME: Sphinx error:
@@ -514,14 +514,14 @@ jobs:
514514
uses: wamp-proto/wamp-cicd/actions/upload-artifact-verified@main
515515
with:
516516
name: flatbuffers-gen-${{ matrix.python-env }}
517-
path: src/autobahn/wamp/gen/
517+
path: ${{ github.workspace }}/src/autobahn/wamp/gen/
518518
retention-days: 7
519519

520520
- name: Upload FlatBuffers schema files
521521
uses: wamp-proto/wamp-cicd/actions/upload-artifact-verified@main
522522
with:
523523
name: flatbuffers-schema-${{ matrix.python-env }}
524-
path: src/autobahn/wamp/flatbuffers/
524+
path: ${{ github.workspace }}/src/autobahn/wamp/flatbuffers/
525525
retention-days: 7
526526

527527
build-package:
@@ -595,5 +595,5 @@ jobs:
595595
uses: wamp-proto/wamp-cicd/actions/upload-artifact-verified@main
596596
with:
597597
name: package-${{ matrix.python-env }}
598-
path: dist/
598+
path: ${{ github.workspace }}/dist/
599599
retention-days: 14

.github/workflows/wheels-arm64.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -424,7 +424,7 @@ jobs:
424424
uses: wamp-proto/wamp-cicd/actions/upload-artifact-verified@main
425425
with:
426426
name: artifacts-arm64-${{ matrix.target.name }}
427-
path: wheelhouse/
427+
path: ${{ github.workspace }}/wheelhouse/
428428
retention-days: 30
429429

430430
# GitHub Releases, PyPI, and RTD publishing are now handled by the centralized 'release' workflow

.github/workflows/wheels-docker.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -480,7 +480,7 @@ jobs:
480480
uses: wamp-proto/wamp-cicd/actions/upload-artifact-verified@main
481481
with:
482482
name: artifacts-${{ matrix.target.name }}
483-
path: wheelhouse/
483+
path: ${{ github.workspace }}/wheelhouse/
484484
retention-days: 30
485485

486486
# GitHub Releases, PyPI, and RTD publishing are now handled by the centralized 'release' workflow

.github/workflows/wheels.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -604,7 +604,7 @@ jobs:
604604
uses: wamp-proto/wamp-cicd/actions/upload-artifact-verified@main
605605
with:
606606
name: wheels-${{ matrix.platform }}-${{ matrix.arch }}
607-
path: dist/
607+
path: ${{ github.workspace }}/dist/
608608
retention-days: 30
609609

610610
- name: Verify wheels built without NVX (Linux x86_64 only)
@@ -787,15 +787,15 @@ jobs:
787787
uses: wamp-proto/wamp-cicd/actions/upload-artifact-verified@main
788788
with:
789789
name: source-distribution
790-
path: dist/
790+
path: ${{ github.workspace }}/dist/
791791
retention-days: 30
792792

793793
- name: Upload Linux wheels without NVX with cryptographic verification (Linux x86_64 only)
794794
if: matrix.platform == 'linux' && matrix.arch == 'x86_64'
795795
uses: wamp-proto/wamp-cicd/actions/upload-artifact-verified@main
796796
with:
797797
name: linux-wheels-no-nvx
798-
path: dist/
798+
path: ${{ github.workspace }}/dist/
799799
retention-days: 30
800800

801801
# GitHub Releases, PyPI, and RTD publishing are now handled by the centralized 'release' workflow

.github/workflows/wstest.yml

Lines changed: 7 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -238,7 +238,7 @@ jobs:
238238
if: always()
239239
with:
240240
name: clients-all-${{ env.TEST_MODE }}-with-nvx
241-
path: .wstest/clients-with-nvx/
241+
path: ${{ github.workspace }}/.wstest/clients-with-nvx/
242242
retention-days: 30
243243

244244
- name: Upload client reports (without-nvx)
@@ -247,7 +247,7 @@ jobs:
247247
if: always()
248248
with:
249249
name: clients-all-${{ env.TEST_MODE }}-without-nvx
250-
path: .wstest/clients-without-nvx/
250+
path: ${{ github.workspace }}/.wstest/clients-without-nvx/
251251
retention-days: 30
252252

253253
# Server testing: Test all server combinations against testsuite client
@@ -481,7 +481,7 @@ jobs:
481481
if: always()
482482
with:
483483
name: servers-all-${{ env.TEST_MODE }}-with-nvx
484-
path: .wstest/servers-with-nvx/
484+
path: ${{ github.workspace }}/.wstest/servers-with-nvx/
485485
retention-days: 30
486486

487487
- name: Upload server reports (without-nvx)
@@ -490,7 +490,7 @@ jobs:
490490
if: always()
491491
with:
492492
name: servers-all-${{ env.TEST_MODE }}-without-nvx
493-
path: .wstest/servers-without-nvx/
493+
path: ${{ github.workspace }}/.wstest/servers-without-nvx/
494494
retention-days: 30
495495

496496
# Consolidate all reports using the proven justfile recipe
@@ -663,15 +663,15 @@ jobs:
663663
uses: wamp-proto/wamp-cicd/actions/upload-artifact-verified@main
664664
with:
665665
name: websocket-conformance-docs-${{ env.TEST_MODE }}-with-nvx
666-
path: docs/_static/websocket/conformance-with-nvx/
666+
path: ${{ github.workspace }}/docs/_static/websocket/conformance-with-nvx/
667667
retention-days: 30
668668

669669
- name: Upload consolidated reports (without-nvx)
670670
id: upload-docs-without-nvx
671671
uses: wamp-proto/wamp-cicd/actions/upload-artifact-verified@main
672672
with:
673673
name: websocket-conformance-docs-${{ env.TEST_MODE }}-without-nvx
674-
path: docs/_static/websocket/conformance-without-nvx/
674+
path: ${{ github.workspace }}/docs/_static/websocket/conformance-without-nvx/
675675
retention-days: 30
676676

677677
- name: Upload summary report for PR comment
@@ -680,8 +680,7 @@ jobs:
680680
with:
681681
name: conformance-summary-${{ env.TEST_MODE }}
682682
# upload-artifact-verified requires a directory, not a file
683-
# Use relative path to avoid any workspace path resolution issues
684-
path: .audit/
683+
path: ${{ github.workspace }}/.audit/
685684
retention-days: 30
686685

687686
- name: Report summary

0 commit comments

Comments
 (0)