aineniamh / snipit

snipit: summarise snps relative to your reference sequence
GNU General Public License v3.0
148 stars 28 forks source link

Suggestion: reformat the code with black #19

Closed terrycojones closed 1 year ago

terrycojones commented 2 years ago

I would be happy to submit some pull requests for snipit, but I find the code style extremely dense, hard to read and to imitate, and generally discouraging.... Without meaning to sound rude or disrespectful etc., I think it would be a healthy move for the project if you installed black and ran it. The result would produce Python that is much more typical and I think encourage participation. Anyway, just a suggestion.

For example, I thought of a way to preserve the original order of sequences in the input, but I don't want to work on Python that looks like this (my editor really screams at me when I load the files). So I just didn't do it, and leave this comment instead. I hope that's ok. Thanks for snipit!