HighCommander4 wrote:
> What's "ping PR"? Do you mean pinging the LLVM organization?
It just means posting a comment on the PR that says "ping", as a reminder for
reviewers.
(But please only do that if you're the patch author, or, in the case of a patch
author who's not active any more, if
aaronliu0130 wrote:
What's "ping PR"? Do you mean pinging the LLVM organization?
https://github.com/llvm/llvm-project/pull/78491
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
cor3ntin wrote:
PRs need to be reviewed before they can be merged.
Feel free to ping PR once a week (at most) if no one gets to it.
This seem to have flown under the radar. sorry about that
https://github.com/llvm/llvm-project/pull/78491
___
aaronliu0130 wrote:
LLVM has 2.5k PRs to review while working on their own changes
https://github.com/llvm/llvm-project/pull/78491
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
tfninjadoom wrote:
I'm new to the processes of open-source software, so pardon my confusion. I
just read through [this](https://github.com/clangd/clangd/issues/529) issue and
am wondering:
What's preventing this PR from being merged right now? All checks seem to have
passed.
aaronliu0130 wrote:
As mentioned in
https://github.com/clangd/clangd/issues/529#issuecomment-1938730781, they're
[here](https://aur.archlinux.org/cgit/aur.git/tree/hover-doxygen-trunk.patch?h=clangd-opt-git).
Hopefully you can follow [the build
trustytrojan wrote:
@aaronliu0130 Do you know if a patch for this exists?
https://github.com/llvm/llvm-project/pull/78491
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
aaronliu0130 wrote:
Well, the last ready-to-go PR was from September (#65448), so probably a while.
https://github.com/llvm/llvm-project/pull/78491
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
trustytrojan wrote:
When will this ever get merged? Would really like to see `vscode-clangd`
finally be able to render Doxygen comments and/or Markdown comments.
https://github.com/llvm/llvm-project/pull/78491
___
cfe-commits mailing list
https://github.com/tom-anders updated
https://github.com/llvm/llvm-project/pull/78491
>From 640318fcf475a82b5483889795e2bd906b7b3c9c Mon Sep 17 00:00:00 2001
From: Tom Praschan <13141438+tom-and...@users.noreply.github.com>
Date: Sat, 18 Feb 2023 15:54:09 +0100
Subject: [PATCH] [clangd] Support
llvmbot wrote:
@llvm/pr-subscribers-clang
@llvm/pr-subscribers-clangd
Author: Tom Praschan (tom-anders)
Changes
This is in preparation for implementing doxygen parsing, see discussion in
https://github.com/clangd/clangd/issues/529.
(Old Phabricator review:
11 matches
Mail list logo