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

Fix deprecated actions #371

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

dloez
Copy link

@dloez dloez commented Feb 17, 2023

Fix deprecation warning "The set-output command is deprecated and will be disabled soon" on jobs.test.steps.pip-cache by redirecting the output of the echo command to $GITHUB_ACTIONS and by using the shell bash to avoid the command not working on Windows default shell.

At first, I was going to submit the PR by using the shell bash not only in that specific step but after thinking about it I only used it on that specific step as using the bash shell in the tox execution step could break everything.

@dloez dloez mentioned this pull request Feb 17, 2023
Copy link

@Borda Borda left a comment

Choose a reason for hiding this comment

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

yes, this is correct

cc: @hugovk

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