ci: Use GITHUB_OUTPUT envvar instead of set-output command

[arunsathiya]

save-state and set-output commands used in GitHub Actions are deprecated and GitHub recommends using environment files.

This PR updates the usage of set-output to $GITHUB_OUTPUT

Instructions for envvar usage from GitHub docs:

https://docs.github.com/en/actions/using-workflows/workflow-commands-for-github-actions#setting-an-output-parameter

[passy]

Thanks! I've been wanting to update that for a while!

[facebook-github-bot]

@passy has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

[facebook-github-bot]

@arunsathiya has updated the pull request. You must reimport the pull request before landing.

[facebook-github-bot]

@arunsathiya has updated the pull request. You must reimport the pull request before landing.

[arunsathiya]

@passy One minor thing that had come to my notice: had to quote the envvar to retain spaces or special characters/treat as a single argument or value. It's done in 3dfa348. Sorry for any troubles!

[facebook-github-bot]

@passy has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

[passy]

@arunsathiya It looks like something is still off for the Windows build. Could you take a look?

[github-actions]

Pull Request Test Coverage Report for Build 7616232349

Warning: This coverage report may be inaccurate.

We've detected an issue with your CI configuration that might affect the accuracy of this pull request's coverage report.
To ensure accuracy in future PRs, please see these guidelines.
A quick fix for this PR: rebase it; your next report should be accurate.

• 0 of 0 changed or added relevant lines in 0 files are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage decreased (-0.02%) to 51.607%

---

Totals
Change from base Build 7611149134:	-0.02%
Covered Lines:	8272
Relevant Lines:	14822

---

💛 - Coveralls

[facebook-github-bot]

@arunsathiya has updated the pull request. You must reimport the pull request before landing.

[arunsathiya]

It seems that Powershell commands need $env prefix, so this commit should help d8a5f2b, I believe. Watching this space to see how the build works out but it needs an approval first: https://github.com/facebook/flipper/actions/runs/7628655259

[facebook-github-bot]

@arunsathiya has updated the pull request. You must reimport the pull request before landing.

[facebook-github-bot]

@passy has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

[passy]

The windows runner still seems unhappy, unfortunately.

[facebook-github-bot]

@arunsathiya has updated the pull request. You must reimport the pull request before landing.

[arunsathiya]

I remember reviewing a bit two weeks but haven't gotten a chance to look again since.

I'll take another look soon but would like the latest checks to run once.

[facebook-github-bot]

@arunsathiya has updated the pull request. You must reimport the pull request before landing.

[facebook-github-bot]

@arunsathiya has updated the pull request. You must reimport the pull request before landing.

[facebook-github-bot]

@passy has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

[facebook-github-bot]

@passy merged this pull request in 1c2fe52.