-
Notifications
You must be signed in to change notification settings - Fork 132
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
Factor out build job logic into a "run-as-coder" reusable workflow. #205
Conversation
image: ${{inputs.build_image}} | ||
command: | | ||
${{ inputs.build_script }} "${{inputs.compiler_exe}}" "${{inputs.std}}" "${{inputs.gpu_build_archs}}" | ||
|
||
test: | ||
needs: build | ||
if: ${{ !cancelled() && ( needs.build.result == 'success' || needs.build.result == 'skipped' ) && inputs.test_script != '' && inputs.test_image != '' && inputs.run_tests}} |
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.
No change requested.
I am wondering whether we could add another result to the build step, that checks whether we had a 100% sccache hit rate. In that case we can omit running the test step
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.
Maybe we can store it in an environment variable
Description
closes #204
Factors out the logic for cloning the cccl repo into the
/home/coder/cccl
directory and executing a command as the coder user into a reusable workflow.Checklist