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

remove deprecated set-output syntax and update create-or-update-comment #4892

Merged
merged 4 commits into from
Nov 2, 2022

Conversation

lldelisle
Copy link
Contributor

This PR only change the github workflow yaml to avoid the deprecation warnings and use the new syntax: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/

@lldelisle lldelisle changed the title remove deprecated set-output syntax remove deprecated set-output syntax and update create-or-update-comment Nov 1, 2022
@lldelisle
Copy link
Contributor Author

In addition it uses the last version of create-or-update-comment to avoid deprecation warning about node version

.github/workflows/ci.yaml Outdated Show resolved Hide resolved
.github/workflows/ci.yaml Outdated Show resolved Hide resolved
.github/workflows/slash.yaml Outdated Show resolved Hide resolved
Co-authored-by: Nicola Soranzo <nicola.soranzo@gmail.com>
@lldelisle
Copy link
Contributor Author

Thanks @nsoranzo I just discover how github actions work so I did not know that the regular way is not to put minor versions.

@nsoranzo nsoranzo merged commit 08fb2b2 into galaxyproject:master Nov 2, 2022
@nsoranzo
Copy link
Member

nsoranzo commented Nov 2, 2022

Thanks @lldelisle !

@bernt-matthias bernt-matthias mentioned this pull request Nov 17, 2022
8 tasks
bernt-matthias added a commit to bernt-matthias/tools-iuc that referenced this pull request Nov 17, 2022
from

- galaxyproject#4850
- galaxyproject#4892

and fix fork (use IUC master for called WFs, instead of my fork)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants