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

Update build_push.yaml: actions/setup-python@v4 (#490) #494

Closed
wants to merge 3 commits into from

Conversation

bryanpaget
Copy link
Contributor

@bryanpaget bryanpaget commented Jul 27, 2023

update actions/setup-python@v2 to actions/setup-python@v4

update actions/setup-python@v2 to actions/setup-python@v4
* Update build_push.yaml: deprecate set-output

update workflow to use $GITHUB_OUTPUT instead of set-output.

See: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/

* Update build_push.yaml: update syntax

* Update build_push.yaml: update syntax for interpolation

* Update build_push.yaml: change notebook name variable name

* update remaining set-output commands

* Update Makefile: reset set-output commands for Makefile

Github Actions was complaining about set-output in the Makefile but I think we have to leave these lines alone since they are used by Make and not by Github Actions.
@bryanpaget bryanpaget marked this pull request as ready for review August 22, 2023 14:25
@bryanpaget bryanpaget closed this Aug 22, 2023
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.

1 participant