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

tool: Replace set-output with GITHUB_OUTPUT #4155

Merged
merged 1 commit into from
Feb 6, 2023

Conversation

kanata2
Copy link
Contributor

@kanata2 kanata2 commented Feb 6, 2023

What this PR does / why we need it:

I am getting the following warning in pipe-cd/[email protected].
Therefore, we have changed to use GITHUB_OUTPUT environment variable instead of set-output.

Warning: The set-output command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/

Which issue(s) this PR fixes:

N/A

Does this PR introduce a user-facing change?:

NONE

@github-actions
Copy link
Contributor

github-actions bot commented Feb 6, 2023

CLA Assistant Lite bot All Contributors have signed the CLA.

@kanata2
Copy link
Contributor Author

kanata2 commented Feb 6, 2023

/cla sign

Copy link
Member

@khanhtc1202 khanhtc1202 left a comment

Choose a reason for hiding this comment

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

Nice catch, thank you!

@khanhtc1202 khanhtc1202 enabled auto-merge (squash) February 6, 2023 04:16
@khanhtc1202 khanhtc1202 merged commit 187f734 into pipe-cd:master Feb 6, 2023
@kanata2 kanata2 deleted the do-not-use-set-output branch February 6, 2023 04:31
This was referenced Feb 7, 2023
@kanata2
Copy link
Contributor Author

kanata2 commented Feb 7, 2023

@khanhtc1202 Thanks! Could you make a release in pipe-cd/actions-gh-release?

knanao added a commit that referenced this pull request Feb 7, 2023
#4157 #4161 (#4163)

* Templating commit message used by Event watcher (#4120)

* Added ability to teplate eventWatcher.gitRepos.commitMessage

* parse template in the eventWatcher handler side

* Parse template in the eventWatcher handler side

* Fix html/template to text/template

* There was a useless process when retrieving rules, so I omitted it.

* Delete html/template

* Update pkg/app/piped/eventwatcher/eventwatcher.go

Co-authored-by: Khanh Tran <[email protected]>

* Update pkg/app/piped/eventwatcher/eventwatcher.go

Co-authored-by: Khanh Tran <[email protected]>

* Using template variables are only Value and EventName.

* Update pkg/app/piped/eventwatcher/eventwatcher.go

Co-authored-by: knanao <[email protected]>

* Update pkg/app/piped/eventwatcher/eventwatcher.go

Co-authored-by: knanao <[email protected]>

* Fix syntax error

* Update pkg/app/piped/eventwatcher/eventwatcher.go

Co-authored-by: Khanh Tran <[email protected]>

Co-authored-by: Khanh Tran <[email protected]>
Co-authored-by: knanao <[email protected]>

* Add environment variable expire web session (#4127)

* add field session-ttl to project

* edit values

* reflect ttl config to session

* return values

* add field in document

* return config

* change variable name

* modify document and manifest

* modify docs

* change minites tp hours

* regen

* Change build tag every time when make run/pipecd (#4130)

* add unix time to tail of build tag

* modify assigning

* change variable name

* change type pf assign

* Fix docs wrong params name (#4132)

* Fix pod config for envoy upstreaming termination error (#4134)

* add lifecycle

* add preStop to api server's manifest

* return makefile

* Fix Fix parseComitMsg's comment (#4135)

* Add kubectl version to stage log (#4137)

* Add kubectl version to stage log

* Update pkg/app/piped/executor/kubernetes/kubernetes.go

Co-authored-by: knanao <[email protected]>

Co-authored-by: knanao <[email protected]>

* Add pipecd builtin tags to ECS resources (#4140)

* Add tags to aws resources.

* Small fix

* Small fix

* Fix initial slice length.

* Add TagResource function

* Fix deploy process

* Add tags to aws resources.

* Small fix

* Small fix

* Fix initial slice length.

* Add TagResource function

* Fix deploy process

* Delete Unnecessary blank lines

* Change how to add task's tags

* Delete TagResource method

* Fix based on linter

* Apply review

* add builtin-labels

* Add tagResource func

* Set default value of PropagateTags

* Apply review

* Apply review

* Add upgrade a release method for helm (#4143)

* tool: Replace set-output with GITHUB_OUTPUT (#4155)

* Make all changes that not be listed in ignores but placed under appDir considered as touched commit (#4157)

* Release v0.41.4 (#4161)

* Release v0.41.4

* Update the release note

---------

Co-authored-by: funera1 <[email protected]>
Co-authored-by: Khanh Tran <[email protected]>
Co-authored-by: kevin55156 <[email protected]>
Co-authored-by: Tomoki Hori <[email protected]>
Co-authored-by: Naoki Kanatani <[email protected]>
@khanhtc1202
Copy link
Member

khanhtc1202 commented Feb 7, 2023

@kanata2 sorry for the late, we just released a new version v2.5.0 for pipe-cd/actions-gh-release that contains your fix. Thank you!
https://github.com/pipe-cd/actions-gh-release/releases/tag/v2.5.0

@kanata2
Copy link
Contributor Author

kanata2 commented Feb 7, 2023

Thanks!

@knanao
Copy link
Member

knanao commented Feb 7, 2023

@kanata2
Sorry, I've released v2.6.0 for actions-gh-release, so please use it instead of v2.5.0.

@github-actions github-actions bot mentioned this pull request Feb 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants