<div dir="ltr">Hi Avishay,<div><br></div><div>Welcome and thanks for your interest in clangd!</div><div><br></div><div>In addition to Nathan's comments, I would also suggest discussing the approach you want to take before submitting a patch for review.<br></div><div>Especially if the approach can be condensed to a few sentences.</div><div><br></div><div></div><div>Things usually go faster if reviewers and the author are aligned on the direction of the patch beforehand.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Oct 1, 2019 at 4:37 PM Avishay Matayev via clangd-dev <<a href="mailto:clangd-dev@lists.llvm.org">clangd-dev@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>Thank you for the guidance Nate! <br></div><div><br></div><div>I'll do my best to contribute ASAP :)</div><div><br></div><div>Cheers,</div><div>Avishay.<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, 30 Sep 2019 at 20:58, Nathan Ridge <<a href="mailto:zeratul976@hotmail.com" target="_blank">zeratul976@hotmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Avishay,<br>
<br>
Thanks for your interesting in contributing to clangd! Yes, this mailing list is a fine place to ask questions of this sort.<br>
<br>
> the feature I miss the most is the macro referencing<br>
<br>
I would find this useful as well.<br>
<br>
> I glanced a over the source code and I think I can write the feature and <br>
> submit it for review, but I am unsure about the overall procedure. <br>
> Where do I submit it to? Who looks over it and who accepts it?<br>
<br>
The procedures followed for clangd are largely the same as those for the larger LLVM community, which are documented here [1]. One thing that's a bit different is clangd has its own issue tracker on Github [2].<br>
<br>
Patches are submitted to LLVM's Phabricator instance [3]. In terms of who to ask for review, you can look at who has reviewed other recent changes to the files you're touching. However, you can also submit a patch without explicitly setting reviewers. I think the clangd team monitors and will notice all review requests with "[clangd]" in the commit message. <br>
<br>
Hope that helps. Feel free to ask more questions if you have them!<br>
<br>
Regards,<br>
Nate<br>
<br>
[1] <a href="http://llvm.org/docs/DeveloperPolicy.html" rel="noreferrer" target="_blank">http://llvm.org/docs/DeveloperPolicy.html</a><br>
[2] <a href="https://github.com/clangd/clangd/issues/" rel="noreferrer" target="_blank">https://github.com/clangd/clangd/issues/</a><br>
[3] <a href="https://reviews.llvm.org/" rel="noreferrer" target="_blank">https://reviews.llvm.org/</a></blockquote></div>
_______________________________________________<br>
clangd-dev mailing list<br>
<a href="mailto:clangd-dev@lists.llvm.org" target="_blank">clangd-dev@lists.llvm.org</a><br>
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/clangd-dev" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/clangd-dev</a><br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div>Regards,</div><div>Ilya Biryukov</div></div></div></div></div>