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.

Sign up for GitHub

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

Jump to bottom

[CI] Update GitHub Actions workflows #75337

Merged
merged 2 commits into from
Jan 8, 2023
Merged

[CI] Update GitHub Actions workflows #75337

merged 2 commits into from
Jan 8, 2023

Conversation

giordano
Copy link
Member

@giordano giordano commented Jan 8, 2023

No description provided.

@giordano giordano temporarily deployed to stopwatch January 8, 2023 17:07 — with GitHub Actions Inactive
@giordano
Copy link
Member Author

giordano commented Jan 8, 2023 •
edited
Loading

Uhm, we also need to get rid of our uses of set-output.

@giordano giordano temporarily deployed to stopwatch January 8, 2023 21:19 — with GitHub Actions Inactive
@giordano
Copy link
Member Author

giordano commented Jan 8, 2023 •
edited
Loading

Last commit which replaces the use of the deprecated command set-output seems to be working as expected, see for example

This clears all warnings about set-output, compare summaries

DilumAluthge
DilumAluthge approved these changes Jan 8, 2023
View reviewed changes
Copy link
Member

@DilumAluthge DilumAluthge left a comment

Choose a reason for hiding this comment

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

Thanks!!

@giordano giordano merged commit 623f2ff into master Jan 8, 2023
@giordano giordano deleted the mg/update-gha branch January 8, 2023 21:35
@DilumAluthge
Copy link
Member

Thanks @giordano!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ericphanson ericphanson ericphanson approved these changes

@DilumAluthge DilumAluthge DilumAluthge approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@giordano @DilumAluthge @ericphanson