Closes #1257 - Adds commit ordering option

[thewindsofwinter]

Description

Closes #1257 - Adds topological ordering option for the git log command. See the issue for details.

Checklist

• I have followed the guidelines in the Contributing document
• My changes follow the coding style of this project
• My changes build without any errors or warnings
• My changes have been formatted and linted
• My changes include any required corresponding changes to the documentation
• My changes have been rebased and squashed to the minimal number (typically 1) of relevant commits
• My changes have a descriptive commit message with a short title, including a Fixes $XXX - or Closes #XXX - prefix to auto-close the issue that your PR addresses

[eamodio]

@thewindsofwinter Overall this looks great -- thanks for the contribution! I wonder now though, if we should change this setting to be sort of a gitlens.advanced.commitOrdering, which could be one of null (the default), date, author-date, topo, and then each callsite would take an ordering flag. What do you think?

[thewindsofwinter]

@thewindsofwinter Overall this looks great -- thanks for the contribution! I wonder now though, if we should change this setting to be sort of a gitlens.advanced.commitOrdering, which could be one of null (the default), date, author-date, topo, and then each callsite would take an ordering flag. What do you think?

Hello! I'm glad it looks great, and I think in time I should be able to implement the rest in due time. I was wondering if we should merge this (to address laanwj's issue) and then I can create a new issue for what you suggested. School's started up for me so it might take me some time to change the setting, so I thought it might be better to first have this requested feature and then edit the rest later.

Overall it's your call. Thank you for your time, and I'm glad I didn't mess anything up as of yet :D If there's anything I could improve on please let me know, I'm quite new to this!

[eamodio]

No worries if you don't have time. I don't want to merge this in directly and then have to change the settings, but I can add on to this PR to add those features, and then merge it. Thanks again for your contribution!!

[thewindsofwinter]

Alright, thank you for your time in reviewing this!

[Shashank-Shastri]

@eamodio I updated @thewindsofwinter code and added some more code for the option you mentioned. Please have a look here: #1421. This is my first time contributing, so do let me know if I did something wrong.

Thanks,
Shashank

[thewindsofwinter]

@eamodio I updated @thewindsofwinter code and added some more code for the option you mentioned. Please have a look here: #1421. This is my first time contributing, so do let me know if I did something wrong.

Thanks,
Shashank

Thank you! Sorry I couldn't get to it sooner

[eamodio]

@thewindsofwinter @Shashank-Shastri

Thank you so much for your contributions!