[PATCH] D19848: DeveloperPolicy: Bugreports should be referenced with an URL

Matthias Braun via llvm-commits llvm-commits at lists.llvm.org
Mon May 2 18:47:33 PDT 2016


MatzeB created this revision.
MatzeB added a subscriber: llvm-commits.
MatzeB set the repository for this revision to rL LLVM.
Herald added a subscriber: mcrosier.

Encourage people to use full URLs when referencing problem reports in commit messages. The URL doesn't add much overhead and allows people to click on it when viewing the messages.

Repository:
  rL LLVM

http://reviews.llvm.org/D19848

Files:
  docs/DeveloperPolicy.rst

Index: docs/DeveloperPolicy.rst
===================================================================
--- docs/DeveloperPolicy.rst
+++ docs/DeveloperPolicy.rst
@@ -314,7 +314,9 @@
   code snippets and gory details should be left to bug comments, web
   review or the mailing list.
 
-* If the patch fixes a bug in bugzilla, please include the PR# in the message.
+* If the patch fixes a bug in bugzilla, please include a reference to
+  http://llvm.org/PR# in the message. Use a full URL rather than just the
+  PR number.
 
 * `Attribution of Changes`_ should be in a separate line, after the end of
   the body, as simple as "Patch by John Doe.". This is how we officially
@@ -327,7 +329,7 @@
 * If the commit is a bug fix on top of another recently committed patch, or a
   revert or reapply of a patch, include the svn revision number of the prior
   related commit. This could be as simple as "Revert rNNNN because it caused
-  PR#".
+  http://llvm.org/PR#".
 
 For minor violations of these recommendations, the community normally favors
 reminding the contributor of this policy over reverting. Minor corrections and


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D19848.55933.patch
Type: text/x-patch
Size: 1122 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160503/3ffed5f2/attachment.bin>


More information about the llvm-commits mailing list