Skip to content

Commit 7e3ad64

Browse files
Replace deprecated ::set-output with $GITHUB_OUTPUT (#1361)
## Summary - Replace deprecated `::set-output` workflow command with the recommended `>> $GITHUB_OUTPUT` syntax in `.github/workflows/integration-tests.yml` - The `::set-output` command was [deprecated by GitHub](https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/) due to a log injection vulnerability (untrusted data could be injected into the output) - The new `$GITHUB_OUTPUT` environment file approach is the recommended replacement ## Test plan - [ ] Verify the `check_secrets` job still correctly sets the `has_token` output - [ ] Verify downstream jobs that depend on `has_token` output continue to work NO_CHANGELOG=true This pull request was AI-assisted by Isaac. Signed-off-by: Hector Castejon Diaz <hector.castejon@databricks.com>
1 parent b2d34e5 commit 7e3ad64

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

.github/workflows/integration-tests.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -25,10 +25,10 @@ jobs:
2525
run: |
2626
if [ -z "${{ secrets.DECO_WORKFLOW_TRIGGER_APP_ID }}" ] || [ -z "${{ secrets.DECO_TEST_APPROVAL_APP_ID }}" ]; then
2727
echo "Required secrets are missing. User has no access to secrets."
28-
echo "::set-output name=has_token::false"
28+
echo "has_token=false" >> $GITHUB_OUTPUT
2929
else
3030
echo "All required secrets are set. User has access to secrets."
31-
echo "::set-output name=has_token::true"
31+
echo "has_token=true" >> $GITHUB_OUTPUT
3232
fi
3333
3434
trigger-tests:

0 commit comments

Comments
 (0)