1925668 - [wpt-sync] Sync PR 48704 - Fix strings within GitHub Actions workflow expressions

Status: RESOLVED FIXED

(Testing :: web-platform-tests, task, P4)

Description

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Sync web-platform-tests PR 48704 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/48704
Details from upstream follow.

Sam Sneddon <[e-mail]> wrote:

Fix strings within GitHub Actions workflow expressions

https://docs.github.com/en/actions/writing-workflows/choosing-what-your-workflow-does/evaluate-expressions-in-workflows-and-actions#literals:

You don't need to enclose strings in ${{ and }}. However, if you do,
you must use single quotes (') around the string. To use a literal
single quote, escape the literal single quote using an additional
single quote (''). Wrapping with double quotes (") will throw an
error.

And in a shocking turn of events, using double quotes did in fact throw an error.

Comment 1

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

PR 48704 applied with additional changes from upstream: b4847226b3e278f0daea1f22d137fb2da1bb50b6, 7e91324675c24957fa5bdf82b16f0c337754cb17

Comment 2

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Pushed to try https://treeherder.mozilla.org/#/jobs?repo=try&revision=d3b6a98fad4e45bc0082482607c64af3e999c4cc

Comment 3

[Web Platform Test Sync Bot [:wpt-sync] (Matrix: #interop:mozilla.org)]

Test result changes from PR not available.

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/4a7fa909fe25
[wpt PR 48704] - Fix strings within GitHub Actions workflow expressions, a=testonly

Comment 5

[Cosmin Sabou [:CosminS]]

https://hg.mozilla.org/mozilla-central/rev/4a7fa909fe25