Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(352)

Issues Search
    My Issues | Starred     Open | Closed | All

Side by Side Diff: man/html/git-hyper-blame.html

Issue 1629253002: git hyper-blame: Added approx. line number translation. (Closed) Base URL: https://chromium.googlesource.com/chromium/tools/depot_tools.git@git-hyper-blame
Patch Set: Added Caveats section to man page. Created 4 years, 10 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
« no previous file with comments | « git_hyper_blame.py ('k') | man/man1/git-hyper-blame.1 » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.1//EN" 1 <!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.1//EN"
2 "http://www.w3.org/TR/xhtml11/DTD/xhtml11.dtd"> 2 "http://www.w3.org/TR/xhtml11/DTD/xhtml11.dtd">
3 <html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en"> 3 <html xmlns="http://www.w3.org/1999/xhtml" xml:lang="en">
4 <head> 4 <head>
5 <meta http-equiv="Content-Type" content="application/xhtml+xml; charset=UTF-8" / > 5 <meta http-equiv="Content-Type" content="application/xhtml+xml; charset=UTF-8" / >
6 <meta name="generator" content="AsciiDoc 8.6.9" /> 6 <meta name="generator" content="AsciiDoc 8.6.9" />
7 <title>git-hyper-blame(1)</title> 7 <title>git-hyper-blame(1)</title>
8 <style type="text/css"> 8 <style type="text/css">
9 /* Shared CSS for AsciiDoc xhtml11 and html5 backends */ 9 /* Shared CSS for AsciiDoc xhtml11 and html5 backends */
10 10
(...skipping 802 matching lines...) Expand 10 before | Expand all | Expand 10 after
813 134200d1 (lorem 2014-04-10 08:54:46 +0000 4*) ALIQUA. UT ENIM AD MINIM VENIAM, Q UIS 813 134200d1 (lorem 2014-04-10 08:54:46 +0000 4*) ALIQUA. UT ENIM AD MINIM VENIAM, Q UIS
814 c6eb3bfa (lorem 2014-08-11 23:15:57 +0000 5) NOSTRUD EXERCITATION ULLAMCO LABOR IS 814 c6eb3bfa (lorem 2014-08-11 23:15:57 +0000 5) NOSTRUD EXERCITATION ULLAMCO LABOR IS
815 0f0d17bd (dolor 2014-06-02 11:31:48 +0000 6*) NISI UT ALIQUIP EX EA COMMODO CONS EQUAT. 815 0f0d17bd (dolor 2014-06-02 11:31:48 +0000 6*) NISI UT ALIQUIP EX EA COMMODO CONS EQUAT.
816 </code></pre></div></div><p><div class="paragraph"></p></div> 816 </code></pre></div></div><p><div class="paragraph"></p></div>
817 <div class="paragraph"><p><code>hyper-blame</code> places a <code>*</code> next to any line where it has skipped over an ignored 817 <div class="paragraph"><p><code>hyper-blame</code> places a <code>*</code> next to any line where it has skipped over an ignored
818 commit, so you know that the line in question has been changed (by an ignored 818 commit, so you know that the line in question has been changed (by an ignored
819 commit) since the given person wrote it.</p></div> 819 commit) since the given person wrote it.</p></div>
820 </div> 820 </div>
821 </div> 821 </div>
822 <div class="sect1"> 822 <div class="sect1">
823 <h2 id="_caveats">CAVEATS</h2>
824 <div class="sectionbody">
825 <div class="paragraph"><p>When a line skips over an ignored commit, a guess is m ade as to which commit
826 previously modified that line, but it is not always clear where the line came
827 from. If the ignored commit makes lots of changes in close proximity, in
828 particular adding/removing/reordering lines, then the wrong authors may be
829 blamed for nearby edits.</p></div>
830 <div class="paragraph"><p>For this reason, <code>hyper-blame</code> works best w hen the ignored commits are be
831 limited to minor changes such as formatting and renaming, not refactoring or
832 other more invasive changes.</p></div>
833 </div>
834 </div>
835 <div class="sect1">
823 <h2 id="_bugs">BUGS</h2> 836 <h2 id="_bugs">BUGS</h2>
824 <div class="sectionbody"> 837 <div class="sectionbody">
825 <div class="ulist"><ul> 838 <div class="ulist"><ul>
826 <li> 839 <li>
827 <p> 840 <p>
828 When a commit is ignored, hyper-blame currently just blames the same line in
829 the previous version of the file. This can be wildly inaccurate if the ignored
830 commit adds or removes lines, resulting in a completely wrong commit being
831 blamed.
832 </p>
833 </li>
834 <li>
835 <p>
836 There is currently no way to pass the ignore list as a file. 841 There is currently no way to pass the ignore list as a file.
837 </p> 842 </p>
838 </li> 843 </li>
839 <li> 844 <li>
840 <p> 845 <p>
841 It should be possible for a git repository to configure an automatic list of 846 It should be possible for a git repository to configure an automatic list of
842 commits to ignore (like <code>.gitignore</code>), so that project owners can m aintain a 847 commits to ignore (like <code>.gitignore</code>), so that project owners can m aintain a
843 list of "big change" commits that are ignored by hyper-blame by default. 848 list of "big change" commits that are ignored by hyper-blame by default.
844 </p> 849 </p>
845 </li> 850 </li>
(...skipping 11 matching lines...) Expand all
857 <div class="sectionbody"> 862 <div class="sectionbody">
858 <div class="paragraph"><p>Part of the chromium <a href="depot_tools.html">depot_ tools(7)</a> suite. These tools are meant to 863 <div class="paragraph"><p>Part of the chromium <a href="depot_tools.html">depot_ tools(7)</a> suite. These tools are meant to
859 assist with the development of chromium and related projects. Download the tools 864 assist with the development of chromium and related projects. Download the tools
860 from <a href="https://chromium.googlesource.com/chromium/tools/depot_tools.git"> here</a>.</p></div> 865 from <a href="https://chromium.googlesource.com/chromium/tools/depot_tools.git"> here</a>.</p></div>
861 </div> 866 </div>
862 </div> 867 </div>
863 </div> 868 </div>
864 <div id="footnotes"><hr /></div> 869 <div id="footnotes"><hr /></div>
865 <div id="footer"> 870 <div id="footer">
866 <div id="footer-text"> 871 <div id="footer-text">
867 Last updated 2016-01-28 16:40:21 AEDT 872 Last updated 2016-02-05 13:43:52 AEDT
868 </div> 873 </div>
869 </div> 874 </div>
870 </body> 875 </body>
871 </html> 876 </html>
OLDNEW
« no previous file with comments | « git_hyper_blame.py ('k') | man/man1/git-hyper-blame.1 » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698