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

ci: fix windows PATH env #241

Merged
merged 1 commit into from
Jul 31, 2024

Conversation

ocavue
Copy link
Contributor

@ocavue ocavue commented Jul 31, 2024

I believe echo "PATH=$LIBTORCH/lib;$PATH" >> $env:GITHUB_ENV breaks the PATH on Windows, and causes milliewalky/setup-7-zip@v1 not working (i.e. cannot find curl). This PR fixes the issue by using the GITHUB_PATH (not GITHUB_ENV) for adding PATH.

Docs: https://docs.github.com/en/actions/writing-workflows/choosing-what-your-workflow-does/workflow-commands-for-github-actions#adding-a-system-path

Copy link

Code Metrics Report
  ===============================================================================
 Language            Files        Lines         Code     Comments       Blanks
===============================================================================
 TOML                    1           70           58            2           10
-------------------------------------------------------------------------------
 Rust                   66        14511        12415          201         1895
 |- Markdown            35          343            0          272           71
 (Total)                          14854        12415          473         1966
===============================================================================
 Total                  67        14581        12473          203         1905
===============================================================================
  

@FL33TW00D
Copy link
Collaborator

🦸🏻

@FL33TW00D FL33TW00D merged commit 27b753b into huggingface:feature/acceptance Jul 31, 2024
1 check passed
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