[llvm] [workflows] Update commit access request PR links (PR #146169)
Jacob Lalonde via llvm-commits
llvm-commits at lists.llvm.org
Fri Jun 27 15:31:03 PDT 2025
================
@@ -297,9 +297,11 @@ def run(self) -> bool:
print(e)
continue
+ total_prs_url = f"https://github.com/llvm/llvm-project/pulls?q=author%3A{self.issue.user.login}+is%3Apr"
+ merged_prs_url = total_prs_url + "+is%3Amerged"
comment = f"""
### Activity Summary:
- * [{total_prs} Pull Requests](https://github.com/llvm/llvm-project/pulls/{self.issue.user.login}) ({merged_prs} merged)
+ * [{total_prs} Pull Requests]({total_prs_url}) ({merged_prs} [merged]({merged_prs_url}))
----------------
Jlalond wrote:
At this point, I think dropping merged onto it's own line makes sense, as I think we're overly focusing on total PR's when we give out access after three regions.
https://github.com/llvm/llvm-project/pull/146169
More information about the llvm-commits
mailing list