Bug #5099
closedCode Cleanup patch
0%
Description
Hey there,
I just inherited a pile of radios and started playing with CHIRP. I noticed that some of the code style is showing it's age and I went ahead and did a general cleanup. They only thing preventing this from running on Python3 is the use of unicode() in repeaterbook.py, platform.py and pyPEG.py. I decided to take a break and show the work I've done to make sure this is desired.
I am attaching a patch file here but did the work on a Github mirror of the code (https://github.com/Cabalist/chirp) I know you all use mercury and not Github but I couldn't find a way to open a pull request here. Let me know if you have any questions or even if this is wanted.
Thanks! I really appreciate that this exists. :)
Files
Updated by Jim Unroe about 7 years ago
- Status changed from New to Feedback
This "CHIRP Development Process":http://chirp.danplanet.com/projects/chirp/wiki/DevelopersProcess page is pretty much what I have been referring to since 2012.
Jim KC9HI
Updated by Bernhard Hailer over 4 years ago
Hi Ryan, I hope you have subscribed to the developers email list since. If you haven't, please check the Wiki here as well: Developers.
Let us know whether you would like to keep the ticket open until your patches make it, or whether we can close it. Be aware that there are efforts underways to make things run on Python 3 already.
Updated by Bernhard Hailer over 4 years ago
- Status changed from Feedback to Closed
- Target version set to chirp-legacy
No more traffic on this ticket.