Bug 476090 - Show the number of changed lines of a commit
Summary: Show the number of changed lines of a commit
Status: NEW
Alias: None
Product: EGit
Classification: Technology
Component: UI (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows 7
: P3 enhancement (vote)
Target Milestone: ---   Edit
Assignee: Project Inbox CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-08-28 03:11 EDT by Akif Etkue CLA
Modified: 2016-09-13 04:24 EDT (History)
4 users (show)

See Also:


Attachments
Example from Gerrit (3.58 KB, image/png)
2015-08-28 03:11 EDT, Akif Etkue CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Akif Etkue CLA 2015-08-28 03:11:13 EDT
Created attachment 256205 [details]
Example from Gerrit

Hallo,

I want to show the number of the changed lines of a commit. 
Similar like in the web UI of Gerrit (see attachment). 

Can someone help me to implement this feature for EGit/JGit.
Comment 1 Mickael Istria CLA 2015-08-28 05:13:18 EDT
Where would you like to see that? In the history view? If so, would it be on the list of commits, or just on commit details part?
Comment 2 Akif Etkue CLA 2015-08-28 08:30:21 EDT
(In reply to Mickael Istria from comment #1)
> Where would you like to see that? In the history view? 

Yes, IMHO history view would be great. 

> If so, would it be on the list of commits, or just on commit details part?

I think the commit list would be better - because better overview. 
So you can see here the big commits and small commits. 

I'm not sure, what you think?
Comment 3 Matthias Sohn CLA 2015-08-28 10:06:16 EDT
moving this feature request to UI component

I think this could be displayed in a new column in the history view's commit list.
The view menu should allow to configure if this column is shown or not (similar to email address).
Comment 4 Akif Etkue CLA 2015-08-28 11:15:07 EDT
(In reply to Matthias Sohn from comment #3)
> I think this could be displayed in a new column in the history view's commit
> list.

Yes, that would be great. 

> The view menu should allow to configure if this column is shown or not

Yes, the column should be configurable.
Comment 5 Akif Etkue CLA 2015-09-11 07:12:35 EDT
(In reply to Matthias Sohn from comment #3)
> moving this feature request to UI component

Would it make sense to add this code to JGit? 
As a part of RevCommit.  

API example:

RevCommit commit = getCommitFromRepo(object);
int numberChangedLines = commit.getNumberOfChangedLines();
Comment 6 Timur Achmetow CLA 2015-11-13 18:18:56 EST
(In reply to Akif Etkue from comment #5)
> Would it make sense to add this code to JGit? 
> As a part of RevCommit.  
> 
> API example:
> 
> RevCommit commit = getCommitFromRepo(object);
> int numberChangedLines = commit.getNumberOfChangedLines();

IMHO is that a nice idea. 
What do you think guys?
Comment 7 Andre Marbeck CLA 2016-09-13 04:24:11 EDT
This is exactly what i'm looking for.
Thumbs up.