github: replace deprecated command with environment file

[jongwooo]

Description

Resolve #6416

Update .github/workflows/release.yml to use environment file instead of deprecated set-output command.
For more information, see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/

I found the workflow file that use set-output command through the following command:

$ find . -name '*.yml' -o -name '*.yaml' | xargs egrep '\bset-output\b'

AS-IS

echo ::set-output name=name::${PACKAGE_NAME}

TO-BE

echo "name=${PACKAGE_NAME}" >> $GITHUB_OUTPUT

RELEASE NOTES: none

[github-actions]

This PR is labeled as requiring an update from the reporter, and no update has been received after 6 days. If no update is provided in the next 7 days, this issue will be automatically closed.

[easwars]

In the blog post you linked to: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/, I see the following example

- name: Set output
run: echo "{name}={value}" >> $GITHUB_OUTPUT

So, should this be:

echo "{name}={PACKAGE_NAME}" >> $GITHUB_OUTPUT

Also, can you verify that your change works?

[jongwooo]

Also, can you verify that your change works?

@easwars Thanks for the review! This issue was also present in grpc-kotlin and has been merged.
I found an example code in GitHub Docs, you can set the name and value as below:

- name: Set color
  id: random-color-generator
  run: echo "SELECTED_COLOR=green" >> "$GITHUB_OUTPUT"
- name: Get color
  run: echo "The selected color is ${{ steps.random-color-generator.outputs.SELECTED_COLOR }}"

[easwars]

Removing the stale label since I had forgotten to publish my comment. Giving the author more time to respond to this.

[dfawley]

So, it's worse than just this. I tested this change, and it works, but it turns out actions/upload-release-asset@v1 also uses set-output and is deprecated. They recommend using a third-party action instead: softprops/action-gh-release

[dfawley]

This should be fine as-is, but it seems there's more work to do if they get rid of set-output entirely