search

mal-lang / mal-toolbox

Apache License 2.0
4 stars 2 forks source link

Standardize order of imports #61

Open mrkickling opened 4 months ago

mrkickling commented 4 months ago 
          For the order of imports we can standardize what goes first with isort in CI and sticking to a convention.

_Originally posted by @nkakouros in https://github.com/mal-lang/mal-toolbox/pull/53#discussion_r1637095298_

andrewbwm commented 4 months ago

This one surprises me a little bit because I tried to follow the PEP-8 standard. How many instances are there where this is not the case?

Maybe it's just that I mistook something for a standard library.

nkakouros commented 4 months ago

This was a comment I made to a commit where @mrkickling manually moved import statements around, and I suggested we automate this with isort for less cognitive load.