Skip to content

Commit

Permalink
Fix Safari artifact names
Browse files Browse the repository at this point in the history
With the unified workflow, we don't actually need to specify the
wildcard, as this will just lead to errors like:

> Error: The artifact name is not valid:
> safari-technology-preview-results-*-6. Contains the following
> character: Asterisk *

e.g., in
https://github.com/web-platform-tests/wpt/actions/runs/11410328782/job/31752427266
  • Loading branch information
gsnedders committed Oct 18, 2024
1 parent 2ab788d commit c7805fe
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/safari_stable.yml
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,6 @@ jobs:
github.event_name != 'workflow_run' || fromJSON(needs.check-workflow-run.outputs.updated-refs)[0] != null
uses: ./.github/workflows/safari-wptrunner.yml
with:
artifact-name: "safari-results-*"
artifact-name: "safari-results"
safari-technology-preview: false
safaridriver-diagnose: false
2 changes: 1 addition & 1 deletion .github/workflows/safari_technology_preview.yml
Original file line number Diff line number Diff line change
Expand Up @@ -31,6 +31,6 @@ jobs:
github.event_name != 'workflow_run' || fromJSON(needs.check-workflow-run.outputs.updated-refs)[0] != null
uses: ./.github/workflows/safari-wptrunner.yml
with:
artifact-name: "safari-technology-preview-results-*"
artifact-name: "safari-technology-preview-results"
safari-technology-preview: true
safaridriver-diagnose: false

0 comments on commit c7805fe

Please sign in to comment.