Charlie Marsh
Charlie Marsh
Also removing syntax error in doctest.
(Doing F405 now.)
Yeah we can revisit F811... I can see the usefulness. It's a bit tricky to get right and not a huge value-add so wanted to cut for 0.1.0.
If they're important to you, they can be! I thought those were made obsolete by Black, but if you need E302 and W292 I'm happy to implement.
No problem at all! Right now, it's more of a prioritization decision than a strictly philosophical one. I'm not opposed to including stylistic rules, I just intentionally deprioritized those rules...
@harupy - Do you plan to do the last few checks here? (N806, N816, N818) No pressure, just trying to keep tabs on the status of the issue.
@harupy - Awesome :)
Yeah it makes some appearances in the RustPython AST but it doesn't seem to be supported yet by the actual parser.
Alternatively, could we use the CPython AST parser directly? It's written in C, AFAIK it should be very fast (perhaps even faster than the RustPython parser).
_Alternatively_, we could try to continue merging improvements back to RustPython.