[PATCH] D19848: DeveloperPolicy: Bugreports should be referenced with an URL
Matthias Braun via llvm-commits
llvm-commits at lists.llvm.org
Wed May 4 12:56:30 PDT 2016
MatzeB updated this revision to Diff 56190.
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
@@ -326,8 +328,8 @@
* 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#".
+ related commit in the message. This could be as simple as "Revert rNNNN
+ because it caused 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.56190.patch
Type: text/x-patch
Size: 1217 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160504/1eb285a6/attachment.bin>
More information about the llvm-commits
mailing list