Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Action: Replace deprecated set-output command #668

Merged
merged 2 commits into from Oct 31, 2022

Conversation

airtower-luna
Copy link
Contributor

@airtower-luna airtower-luna commented Oct 27, 2022

The set-output workflow command has been deprecated and replaced with writing to an environment file. Update .github/action_helper.py to use the new file.

.github/action_helper.py Outdated Show resolved Hide resolved
@FollowTheProcess
Copy link
Collaborator

This looks good, just needs a conflict resolution because I merged #667 in before this one 👍🏻

The set-output workflow command has been deprecated [1] and replaced
with writing to an environment file. Update .github/action_helper.py
to use the new file.

[1] https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Changing .github/action_helper.py changes the behavior of the action
even if action.yml doesn't change. This makes tests run anyway.
@airtower-luna
Copy link
Contributor Author

Rebased on main.

@airtower-luna
Copy link
Contributor Author

Does anyone have an idea what went wrong in the "macos-latest, 3.8" build? The log looks to me like a KeyboardInterrupt within Nox' process management bubbled up to pytest, but that's code I haven't touched, and it passed CI on main, so I'm confused what I should do. 🤔

@FollowTheProcess
Copy link
Collaborator

Looks like a spurious error to me, I'll re-run and see what happens

@FollowTheProcess
Copy link
Collaborator

Okay, well now it's 3.11 apparently!

@FollowTheProcess
Copy link
Collaborator

5th time's the charm it seems!

@FollowTheProcess FollowTheProcess merged commit e7cf847 into wntrblm:main Oct 31, 2022
@FollowTheProcess
Copy link
Collaborator

Thanks @airtower-luna 🎉

@airtower-luna airtower-luna deleted the replace-set-output branch October 31, 2022 18:07
airtower-luna added a commit to airtower-luna/mpvasync that referenced this pull request Oct 31, 2022
Including switching wntrblm/nox to upstream after my patch has been
merged [1].

[1] wntrblm/nox#668
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants