You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I don't have a ton of free time at the moment, but here are some (opinionated) suggestions for making the repo easier to maintain in the future. Take these as suggestions to be discussed
https://mypy.readthedocs.io/en/stable/ for static typing. I've become a huge proponent of static typing for being able to quickly understand code (especially old code you haven't looked at for a while)
Using click instead of argparse. IMO click is so easy to work with, and a small enough dependency, that it's a better choice.
The text was updated successfully, but these errors were encountered:
@kylebarron I don't have particularly strong feelings on these. Of course I have peeves about formatting but I'm self-aware in that I know they're subjective (and inconsistent Un_n).
pyupgrade seems like a good idea.
black looks good; I would assume moving from YAPF won't be hard.
isort seems fine—I reckon consistency matters more than the actual choices in this case.
click seems nice (I think I have a memory of argparse being annoying to work around when doing the magics but not 100%).
I have mixed feelings on static typing for python. In general I'm quite happy for languages to have static types (or optional static types, like with Julia). In Python's case the interpreter ignores it, AFAIK, which makes me sad ):
So if there's a situation where typing matters the check has to be added by the programmer. While mypy sounds like a good idea nonetheless, I guess it's hard for me to be excited about it when a) I haven't seen it in action and, more importantly, b) migrating to static typing seems by far to be the most labor-intensive bullet point.
I don't have a ton of free time at the moment, but here are some (opinionated) suggestions for making the repo easier to maintain in the future. Take these as suggestions to be discussed
The text was updated successfully, but these errors were encountered: