-
Notifications
You must be signed in to change notification settings - Fork 492
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
Migrate to GitHub Actions #1181
Conversation
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.
Thanks for kicking this off. Left a small suggestion to help fix the pip jobs. Can we also deprecate the conda jobs in this PR if they are now working on GHA?
@vivekmig Can we merge this change, and fix the remaining broken job as a follow-up? We'd like to deprecate the old CircleCI jobs ASAP |
939d818
to
0131546
Compare
0131546
to
939d818
Compare
.github/workflows/deploy.yml
Outdated
repository: pytorch/captum | ||
script: | | ||
sudo chmod -R 777 . | ||
./scripts/install_via_pip.sh -n -d |
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.
For any calls to pip install
within this script or any other script called from these workflows, we probably should add the following option to make the logs more readable: --progress-bar off
.github/workflows/deploy.yml
Outdated
sudo chmod -R 777 . | ||
./scripts/install_via_pip.sh -n -d | ||
|
||
if [[ $CIRCLE_PROJECT_USERNAME == "pytorch" && -z $CI_PULL_REQUEST && -z $CIRCLE_PR_USERNAME ]]; then |
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.
CIRCLE_PROJECT_USERNAME
does not have a set value. Any env vars without a set value with fail with unbound environment variable
since we runs this shell script with set -u
.
d80ea70
to
627e13b
Compare
@vivekmig has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
Migrates test jobs from CircleCI to Github Actions