[llvm] [GitHub] Add basic CI for libclang Python binding unit tests (PR #76784)

Craig Hesling via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 26 20:52:39 PST 2024


================
@@ -0,0 +1,39 @@
+name: Libclang Python Binding Tests
+
+permissions:
+  contents: read
+
+on:
+  workflow_dispatch:
+  push:
----------------
linux4life798 wrote:

Hmmm. What's the issue with the applicable test running in forks?

I intentionally didn't add these restriction to allow for easier testing of changes in forks. I figured if you didn't like a particular workflow, then you just disable the specific workflow using the GitHub UI. No code modified. Plus, I believe all workflows are disabled by default for new forks, as I called out in https://github.com/llvm/llvm-project/pull/76784/commits/8fd0cac97dc03a00044b63e407da9b39f4a5bac4.

The alternative, where we disable them from within the yaml file, doesn't seem great to me. This means that you have to maintain additional commits/changes in every branch of your fork to re-enable this behavior. When you submit a PR, you have to remove these temporary commits. This discourages local CI testing.

https://github.com/llvm/llvm-project/pull/76784


More information about the llvm-commits mailing list