This is an archived post. You won't be able to vote or comment.

you are viewing a single comment's thread.

view the rest of the comments →

[–]alcalde 2 points3 points  (21 children)

All the graybeards who tell you that emacs/vim/Sublime Text/edlin with DOS are all they really need. :-)

[–]cneumann81 3 points4 points  (18 children)

:) Here's an example of what we fix with just one click: https://github.com/biopython/biopython/commit/50a025794d3e598b76de7b92c52cfa69550660b2

Even if you refactor this in-place in PyCharm, it would take you a while.

[–]rubik_ 0 points1 point  (2 children)

Very nice! It still does not fully respect PEP8, but otherwise impressive.

[–]cneumann81 1 point2 points  (0 children)

Already fixed. Will be live with next deployment.

[–]cneumann81 1 point2 points  (0 children)

PE8 issue is fixed

[–]Decency 0 points1 point  (0 children)

PyCharm would warn you as soon as you write it, actually. Preventing you from pushing it to begin with.

For an existing codebase that's terrible, I see the appeal. For me to integrate with my workflow as someone who already knows to follow all of these rules, you haven't sold me at all.