mathialo / bython

Python with braces. Because python is awesome, but whitespace is awful.
MIT License
2.14k stars 58 forks source link

py2by.py now properly translates python to bython, no -i flag required #9

Closed lynnpepin closed 7 years ago

lynnpepin commented 7 years ago

This is an update of py2by.py and the relevant man page.

py2by now adds indentation based off the Python internal tokenizer, meaning it's not reliant on whitespace assumptions. Should be more reliable, and get rid of a lot of (if not all) cases where py2by failed on valid python code.

lynnpepin commented 7 years ago

Sounds good! I was going to go through the source and add Python docstrings, so I'll add that credit too when I go through it.

lynnpepin commented 7 years ago

@mathialo By the way, how would be the best way for me to change this string?

"py2by is a part of Bython v0.3\nMathias Lohne 2017"

Would it be best to just remove names from the -v argument altogether? As more people contribute, this would get pretty long. If you'd like to keep it, I could just attach my name to there.

mathialo commented 7 years ago

For now I don't see it as a problem to have both names in there, especially since your contribution is pretty severe. If there should be a problem with a lot of authors in future, we can review it