[PATCH] D134740: Document use of Co-author-by git tag.

Kristof Beyls via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 28 02:56:58 PDT 2022


kristof.beyls added inline comments.


================
Comment at: llvm/docs/DeveloperPolicy.rst:270
+  tag 'Co-author-by:' to list the additional authors
+  <https://github.blog/2018-01-29-commit-together-with-co-authors/>`_.
+
----------------
danielkiss wrote:
> Thanks!
> maybe a link to the manual instead?
> https://docs.github.com/en/pull-requests/committing-changes-to-your-project/creating-and-editing-commits/creating-a-commit-with-multiple-authors
> 
> Link with an anchor to the section is maybe too much.
> https://docs.github.com/en/pull-requests/committing-changes-to-your-project/creating-and-editing-commits/creating-a-commit-with-multiple-authors#creating-co-authored-commits-on-the-command-line
I've wondered what exactly to link to.
Rather than pointing to a manual on how to add those tags in a github-specific way, I wanted to point to something that explains why co-author-by tags are useful and what value they bring. The github blog post I'm pointing to in the link is the best I've found (after searching for maybe 5 minutes).
So, I think the link to the github blog is slightly better than a link to the github-specific manual page. That being said, I wouldn't be surprised if there is a better page to point to - it's just that I haven't found it.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D134740/new/

https://reviews.llvm.org/D134740



More information about the llvm-commits mailing list