48: Add "number of lines changed" to dashboard view
- Review Board
|Feb. 16, 2014|
It'd be helpful for prioritizing work to have an indication as to the size of an attached diff shown in the dashboard view.
#2 tle****@gmai***** (Google Code) (Is this you? Claim this profile.)
A single-line summary like "8 files added; 2 files deleted; 11 files modified. 182 lines." would be useful, and shouldn't be too hard to generate.
So, this isn't going to be a quick and easy change. Yes, we can generate that info, but the right way to do so would be to process the diff when the user goes to upload it (which has the advantage of verifying that it's not a broken diff before we actually make it available to view), and then store those stats in the FileDiff and DiffSet models. FileDiff should have fields for number of total lines in the file and number of lines added/deleted/modified. From that, we can gather whether the file was completely added or completely deleted. We could then use that info in the dashboard for a column. The column would be pretty long, and I'd like to find a more compact way to represent it, but the hard part will be the above.
+ Add "number of lines changed" to dashboard view
#8 jahidt******@gmai***** (Google Code) (Is this you? Claim this profile.)
so is there anything for the solution on Reviewboard? I also need the number of lines changed for a revision, and it would be quite handy if Reviewboard provided it.
- Confirmed + PendingReview
Fixed in master (7ba68fe). Whoo!
- PendingReview + Fixed