-
Notifications
You must be signed in to change notification settings - Fork 255
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
Add github action support #542
base: main
Are you sure you want to change the base?
Conversation
- name: Set up Docker Buildx | ||
uses: docker/setup-buildx-action@v1 | ||
- uses: docker/setup-buildx-action@v3 |
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.
Is this right that we set up twice?
run: | | ||
echo "matrix=$(python3 .github/workflows/set_matrix.py)" >> $GITHUB_OUTPUT | ||
|
||
build-and-test: |
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.
Is there a way to set the name here to something nicer than the list of files? I'm imagining build-and-test (1/10)
and then first step can be to echo list of files to output for debugging later?
@madrob Thanks for the feedback! I probably won't be able to attend to this PR for a few weeks. Feel free to take make a separate PR and discard this PR. |
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.
Testing here until GHA is enabled on axlearn: samos123#1
The GHA are working except clip_test.py seems to be broken but I suspect it's broken on circleCI too.
I would prefer to merge this with excluding clip_test.py. I or the whoever wrote/knows clip_test.py can do a follow up PR.