-
Notifications
You must be signed in to change notification settings - Fork 8
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
Added appropriate names for the GitHub actions #191
Conversation
sujik18
commented
Feb 6, 2025
- fix Better naming for github action runs #134
- build_wheel.yml -> updated enviornment parameter value
Fix mlcommons#165, Replacement in the initial three files
test name update
Update job key value and name parameter
on pull request,command to reflect the changes
reverted back to pull_request_target
MLCommons CLA bot All contributors have signed the MLCommons CLA ✍️ ✅ |
.github/workflows/build_wheel.yml
Outdated
@@ -14,7 +14,9 @@ jobs: | |||
if: github.repository_owner == 'mlcommons' | |||
name: Build wheel | |||
runs-on: ubuntu-latest | |||
environment: release |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need this for pypi release. Any reason for this change?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The environment field value was causing error,
Value 'release' is not valid
Available expression contexts: github, inputs, vars, needs, strategy, matrix
so i tried using env key to set environment variables