Skip to content
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

Workflow: label run_ci_matrix allows to start the full CI-matrix manually #1539

Closed
wants to merge 18 commits into from

Conversation

mrfh92
Copy link
Collaborator

@mrfh92 mrfh92 commented Jun 21, 2024

This PR is only concerned with our workflow. Sometimes, one would like to check if the CI-matrix fails before one requests review, and similarly, it sometimes happens that you need to re-request review only because you did some bug-fixes w.r.t. the CI-matrix.

This PR allows you to run the full CI-matrix just by setting the label "run_ci_matrix"; unlabeling and labeling again allows to repeat this.

@mrfh92
Copy link
Collaborator Author

mrfh92 commented Jun 21, 2024

@ClaudiaComito @mtar what do you think about this?

@mrfh92 mrfh92 changed the title DO_NOT_MERGE: played around with CI to have a "run CI-matrix on demand" Workflow: label run_ci_matrix allows to start the full CI-matrix manually Jun 21, 2024
@mrfh92 mrfh92 added the PR talk label Jul 1, 2024
@mrfh92
Copy link
Collaborator Author

mrfh92 commented Jul 1, 2024

decided in the PR talk: move to one file with the old CI matrix

@mrfh92
Copy link
Collaborator Author

mrfh92 commented Jul 1, 2024

decided in PR talk: try to trigger by comment, not by label

@mrfh92 mrfh92 removed the PR talk label Jul 1, 2024
now the full CI matrix can be run by commenting "run_ci_matrix"
@mrfh92
Copy link
Collaborator Author

mrfh92 commented Jul 1, 2024

run_ci_matrix

@mrfh92
Copy link
Collaborator Author

mrfh92 commented Jul 1, 2024

run_ci_matrix

@mrfh92
Copy link
Collaborator Author

mrfh92 commented Jul 1, 2024

run_ci_matrix

@mrfh92
Copy link
Collaborator Author

mrfh92 commented Jul 1, 2024

run_ci_again

1 similar comment
@mrfh92
Copy link
Collaborator Author

mrfh92 commented Jul 1, 2024

run_ci_again

@mrfh92
Copy link
Collaborator Author

mrfh92 commented Jul 1, 2024

run_ci_again

@mrfh92
Copy link
Collaborator Author

mrfh92 commented Jul 1, 2024

run_ci_again

@mrfh92
Copy link
Collaborator Author

mrfh92 commented Jul 1, 2024

run_ci_again

@mrfh92
Copy link
Collaborator Author

mrfh92 commented Jul 1, 2024

runcimatrix

@mrfh92
Copy link
Collaborator Author

mrfh92 commented Jul 2, 2024

@mtar @ClaudiaComito for whatever reason, bringing this together in one file does not work

@mrfh92 mrfh92 closed this Jul 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant