Skip to content

Conversation

@boomanaiden154
Copy link
Contributor

The postcommit workflows set to run as cron jobs are difficult to test without landing changes in main. Add a workflow trigger to run them on pull requests when the workflow definition gets modified.

This will also soon be listed in LLVM's CI best practices document soon. llvm/llvm-project#172235

Landing this primarily because it makes it easier to test some changes I want to make soon.

The postcommit workflows set to run as cron jobs are difficult to test
without landing changes in main. Add a workflow trigger to run them on
pull requests when the workflow definition gets modified.

This will also soon be listed in LLVM's CI best practices document soon.
llvm/llvm-project#172235

Landing this primarily because it makes it easier to test some changes I
want to make soon.
@boomanaiden154
Copy link
Contributor Author

@Icohedron and @llvm-beanz to review since I do not have permissions to request reviews here.

@Keenuts Keenuts requested a review from llvm-beanz December 15, 2025 13:56
@Keenuts
Copy link
Contributor

Keenuts commented Dec 15, 2025

Failures are unrelated: this PR is just triggering workflows that are already broken.

@Keenuts Keenuts merged commit 92b3586 into llvm:main Dec 15, 2025
37 of 47 checks passed
@boomanaiden154 boomanaiden154 deleted the permissions-best-practices branch December 15, 2025 17:47
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