Skip to content
New issue

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

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

Already on GitHub? # to your account

fix(github-actions): Remove old output syntax #326

Merged
merged 3 commits into from
Aug 31, 2023

Conversation

stefanhoth
Copy link
Contributor

The syntax this switch covers has been removed quite a while ago and should not occur in the wild anymore. It likely even fails today.

The syntax this switch covers has been removed quite a while ago and should not occur in the wild anymore. It likely even fails today.
Copy link
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for implementing a fix, could you ensure that the test covers your changes if applicable.

@jackton1
Copy link
Member

Nice @stefanhoth looks good.

@jackton1
Copy link
Member

jackton1 commented Aug 31, 2023

@all-contributors please add @stefanhoth for code

@allcontributors
Copy link
Contributor

@jackton1

I've put up a pull request to add @stefanhoth! 🎉

@jackton1 jackton1 added the merge when passing Merge the PR automatically once all status checks have passed label Aug 31, 2023
@jackton1 jackton1 enabled auto-merge (squash) August 31, 2023 01:03
@jackton1 jackton1 merged commit b414bdd into tj-actions:main Aug 31, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
merge when passing Merge the PR automatically once all status checks have passed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants