[llvm] [CI] Make premerge_advisor_explain write comments (PR #166605)
Baranov Victor via llvm-commits
llvm-commits at lists.llvm.org
Fri Nov 7 13:16:39 PST 2025
vbvictor wrote:
> @boomanaiden154 I'm seeing a build failure in a PR build that looks like it might be related to this change?
+1 Also seen problems.
[On Linux](https://github.com/llvm/llvm-project/actions/runs/19181031691/job/54837627481?pr=167020):
```
File "/home/gha/actions-runner/_work/llvm-project/llvm-project/.ci/premerge_advisor_explain.py", line 78, in main
comment = get_comment(
^^^^^^^^^^^^
TypeError: get_comment() missing 2 required positional arguments: 'pr_number' and 'body'
```
[On Windows](https://github.com/llvm/llvm-project/actions/runs/19181031691/job/54837627484?pr=167020) (different error):
```
Traceback (most recent call last):
File "C:\_work\llvm-project\llvm-project\.ci\premerge_advisor_explain.py", line 23, in <module>
def get_comment_id(platform: str, pr: github.PullRequest.PullRequest) -> int | None:
TypeError: unsupported operand type(s) for |: 'type' and 'NoneType'
```
https://github.com/llvm/llvm-project/pull/166605
More information about the llvm-commits
mailing list