src-d / hercules

Gaining advanced insights from Git repository history.
Other
2.63k stars 334 forks source link

Please order `people_sequence` by the person's first commit #304

Closed Asday closed 5 years ago

vmarkovtsev commented 5 years ago

Hi @Asday can you please be more specific - I don't remember a thing about my code now.

Asday commented 5 years ago

After playing with it a bit, I don't think it does what I wanted. When I do labours -m ownership, it shows some newer contributors underneath older ones, whereas I'd prefer the lowest to the highest in the stack to be in order of appearance.

image

As you can see, with the giant smack of SLOC the purple guy made, it could very well look like the pink guy's work was all removed, and also another pink guy started at the same time.

I thought the order of people_sequence determined the stacking order, but I believe I'm wrong.

vmarkovtsev commented 5 years ago

ok, got it

vmarkovtsev commented 5 years ago

@Asday 10.4.0 has been released: https://pypi.org/project/labours/10.4.0