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

Updated CI pipeline to use Github Environment Files. #866

Merged
merged 1 commit into from
Oct 20, 2022

Conversation

MoFtZ
Copy link
Collaborator

@MoFtZ MoFtZ commented Oct 20, 2022

This PR removes usage of the deprecated ::set-output commands - I noticed warnings in the CI builds.

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

If you are using self-hosted runners make sure they are updated to version 2.297.0 or greater.
Do we need to update our Cuda runners?

UPDATE: There is one more set-output warning, coming from the Download action (tracked here).

- Removes usage of deprecated ::set-output commands.
@m4rs-mt
Copy link
Owner

m4rs-mt commented Oct 20, 2022

@pavlovic-ivan pulling you into the conversation about the Cuda runner versions.

@pavlovic-ivan
Copy link
Contributor

Hi @m4rs-mt @MoFtZ . You are currently using Github runners version 2.297.0. This PR introduces version 2.298.2. Also, Nvidia drivers version is 510. Does this help?

@MoFtZ
Copy link
Collaborator Author

MoFtZ commented Oct 20, 2022

Thanks @pavlovic-ivan. That means we can merge this PR without waiting for #859.

@pavlovic-ivan
Copy link
Contributor

Glad to be of help 🙏

@m4rs-mt m4rs-mt merged commit f386305 into m4rs-mt:master Oct 20, 2022
@MoFtZ MoFtZ deleted the feature/ci-environment-files branch October 21, 2022 19:27
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