Page MenuHomePhabricator

DeveloperPolicy: Bugreports should be referenced with an URL
AbandonedPublic

Authored by MatzeB on May 2 2016, 6:47 PM.

Details

Reviewers
None
Summary

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.

Diff Detail

Repository
rL LLVM

Event Timeline

MatzeB updated this revision to Diff 55933.May 2 2016, 6:47 PM
MatzeB retitled this revision from to DeveloperPolicy: Bugreports should be referenced with an URL.
MatzeB updated this object.
MatzeB set the repository for this revision to rL LLVM.
MatzeB added a subscriber: llvm-commits.

I'm on board!

I have no problem with the suggestion for the body, but please leave the
PR# bit in the subject. The PR is the least useful part of any subject
to me and I don't want to devote more space to it.

Philip

MatzeB updated this revision to Diff 56190.May 4 2016, 12:56 PM

I have no problem with the suggestion for the body, but please leave the
PR# bit in the subject. The PR is the least useful part of any subject
to me and I don't want to devote more space to it.

Philip

I fully agree here and the parts I changed talk about the commit message and not the commit title. I'd also say that should be no reason to even mention the PR# in the title. I changed the revert point to explicitly mention that it is about the commit message.

Although I like the change, I proposed something similar in http://lists.llvm.org/pipermail/llvm-dev/2016-May/099752.html and it was rejected.

+Renato, who was opposed the previous patch.

MatzeB abandoned this revision.Jul 20 2016, 8:30 PM

This seems to have stalled, abandoning.