Skip to content

Allow triggering optional CI tests via /cp2k-ci comments#60

Closed
Growl1234 wants to merge 2 commits into
cp2k:masterfrom
Growl1234:master
Closed

Allow triggering optional CI tests via /cp2k-ci comments#60
Growl1234 wants to merge 2 commits into
cp2k:masterfrom
Growl1234:master

Conversation

@Growl1234
Copy link
Copy Markdown
Contributor

PR authors who are also members of cp2k organization can trigger, cancel, or restart optional tests on their own PRs by commenting /cp2k-ci <target>. Limited to testers previously concluded with "neutral" or "cancelled".

I'm not sure if this change is safe and valid, so I submit as a draft. It might be better if there are CI testers here.

@oschuett
Copy link
Copy Markdown
Member

I think this is good idea and it generally looks good.

My plan for this week is to upgrade the Kubernetes cluster and try to migrate from Docker to Podman. So, I'll park this PR for ~1 week. Feel free to ping me if I don't get back to it in >2 weeks.

Btw, I really wish GitHub offered a better way to interact with tools than through comments 🙄

@Growl1234 Growl1234 marked this pull request as ready for review May 6, 2026 12:37
@oschuett oschuett force-pushed the master branch 5 times, most recently from 3cfef45 to d0df7c0 Compare May 14, 2026 17:02
Growl1234 added 2 commits May 16, 2026 23:15
PR authors who are also members of cp2k organization can now trigger, cancel, or restart optional tests on their own PRs by commenting `/cp2k-ci <target>`. Limited to "neutral" or "cancelled" conclusion.
@Growl1234
Copy link
Copy Markdown
Contributor Author

👀

@oschuett
Copy link
Copy Markdown
Member

Sorry for letting this sit for so long. It's now at the top of my todo list.

I'm not quite happy with the code itself tough. So, if it's ok with you, I'd like to take a stab at it myself?

@Growl1234
Copy link
Copy Markdown
Contributor Author

Sure, thank you for taking this over!

@Growl1234 Growl1234 closed this May 21, 2026
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.

2 participants