Skip to content

Comments

Update ci.yml to use GITHUB_OUTPUT environment variable#27125

Closed
bdsoha wants to merge 1 commit intoapache:mainfrom
bdsoha:feature/set-output-deprecation
Closed

Update ci.yml to use GITHUB_OUTPUT environment variable#27125
bdsoha wants to merge 1 commit intoapache:mainfrom
bdsoha:feature/set-output-deprecation

Conversation

@bdsoha
Copy link
Contributor

@bdsoha bdsoha commented Oct 19, 2022

Updated the ci.yml to use $GITHUB_OUTPUT instead of ::set-output which is being deprecated

@bdsoha bdsoha requested review from ashb, kaxil and potiuk as code owners October 19, 2022 04:57
@bdsoha bdsoha mentioned this pull request Oct 23, 2022
Merged
@bdsoha
Copy link
Contributor Author

bdsoha commented Oct 23, 2022

Already merged #27193

@bdsoha bdsoha closed this Oct 23, 2022
@potiuk
Copy link
Member

potiuk commented Oct 25, 2022

Yep. It was much more than that :)

@bdsoha bdsoha deleted the feature/set-output-deprecation branch October 25, 2022 19:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@potiuk potiuk Awaiting requested review from potiuk potiuk is a code owner

@ashb ashb Awaiting requested review from ashb ashb is a code owner

@kaxil kaxil Awaiting requested review from kaxil kaxil is a code owner

1 more reviewer

@auvipy auvipy auvipy approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

area:dev-tools

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@bdsoha @potiuk @auvipy