Skip to content

Commit 29672b6

Browse files
Replace deprecated ::set-output with $GITHUB_OUTPUT (#737)
## Summary - Replace deprecated `::set-output` workflow command with the recommended `>> $GITHUB_OUTPUT` syntax in `.github/workflows/integration-tests.yml` - The `::set-output` command has been deprecated by GitHub and is vulnerable to log injection attacks ([GitHub changelog](https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/)) ## Test plan - [x] Verify the `check-token` job in `integration-tests.yml` correctly sets the `has_token` output using the new syntax - [x] Confirm downstream jobs that depend on `steps.set-token-status.outputs.has_token` 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 90e354f commit 29672b6

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
@@ -23,10 +23,10 @@ jobs:
2323
run: |
2424
if [ -z "${{ secrets.DECO_WORKFLOW_TRIGGER_APP_ID }}" ] || [ -z "${{ secrets.DECO_TEST_APPROVAL_APP_ID }}" ]; then
2525
echo "Required secrets are missing. User has no access to secrets."
26-
echo "::set-output name=has_token::false"
26+
echo "has_token=false" >> $GITHUB_OUTPUT
2727
else
2828
echo "All required secrets are set. User has access to secrets."
29-
echo "::set-output name=has_token::true"
29+
echo "has_token=true" >> $GITHUB_OUTPUT
3030
fi
3131
3232
trigger-tests:

0 commit comments

Comments
 (0)