src-d / hercules

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

Labours fixes #314

Closed akx closed 5 years ago

akx commented 5 years ago
vmarkovtsev commented 5 years ago

BTW, the DCO check can be fixed with this: https://github.com/src-d/guide/blob/master/developer-community/fix-DCO.md

akx commented 5 years ago

Thank you for finding the courage for improving this ~crappy~ out-of-hand Python code!

No problem! How would you feel about a future PR that'd refactor labours into a package instead of one monolithic file? I don't mind doing the work.

vmarkovtsev commented 5 years ago

How would you feel about a future PR that'd refactor labours into a package instead of one monolithic file?

That would be amazing. The package is already there, we just need to split into different files.