[llvm] ci: Fix `pr-code-format` permissions for private forks (PR #120838)

via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 25 05:18:40 PST 2024


whisperity wrote:

> Also, running the LLVM workflows in forks is not really a well-supported use case currently.
> 
> People hacking on the actions typically make sure they work in the monorepo at head and leave it at that.

In that case, should I create a patch that restricts the running of the action `if: github.repository_owner == 'llvm'` like it is done for many of the other workflow files?

> If people want to maintain this and the changes are simple enough and don't impact how they operate in the monorepo

It is very unlikely that anything would be changed by such a change.

@tstellar Can you (or someone who has the rights) check how LLVM organisation is set up? Likely the write permission isn't needed in

> The comments are added by this workflow instead: https://github.com/llvm/llvm-project/blob/main/.github/workflows/issue-write.yml

either, because [the org is set to allow write rights implicitly](https://github.com/organizations/llvm/settings/actions)?
![image](https://github.com/user-attachments/assets/e4f45c0d-4f8c-4289-b504-e2cd3332a3b1)


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


More information about the llvm-commits mailing list