-
Notifications
You must be signed in to change notification settings - Fork 3
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: delegate cdash status into the cdash-status action #38
ci: delegate cdash status into the cdash-status action #38
Conversation
eba7ff2
to
80e71ba
Compare
80e71ba
to
372ffa6
Compare
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.
Co-authored-by: Kenneth Moreland <[email protected]>
@kmorel we do not have the cdash status since I removed the Circle CI infrastructure code. The reason of the new workflow not running now is that this workflow runs in |
@vicentebolea I only understood the first sentence of that. So, where do you go to find the CDash reporting for a PR? |
We can only test this kind of github workflow once it is merged in main, this is by design of |
OK. So, you are saying that after this is merged in, we will see the CDash link in the list of checks. If that is the case, then +1. |
I have recently created a github action that setup the cdash status as shown here: https://github.com/marketplace/actions/cdash-status
In this PR remove the adhoc implementation of the cdash status and delegate into this action. This will help me not to duplicate effort across the different project that uses this.