This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author Lita.Cho
Recipients Lita.Cho, gregorlingl, jesstess, terry.reedy
Date 2014-05-30.20:46:49
SpamBayes Score -1.0
Marked as misclassified Yes
Message-id <1401482811.58.0.970449085174.issue21573@psf.upfronthosting.co.za>
In-reply-to
Content
Hi Terry,

> is there project link? are any of the mentors core developers, with
>  commit rights? or would you need commits from someone like me?

I am not 100% sure. Let me ask Jessica, who is my mentor, and get back to you.

> I have read turtle.py and found that the multiple layers made some 
> things hard to follow. To me, this was a bigger impediment than code
>  formatting. You may want to develop an outline of the layer
>  structure.

There is no project link to Turtle cleanup specifically. But I can definitely try to reorganize the code such that it flows better and it is easier to read. I can add that to the list.

> As you may know, Guido generally discourages pure code cleanups and
> prefers that they be done when the code is examined for other 
> purposes. I personally think some changes are safe enough if verified 
> by a second person.

I actually did not know that Guido discourages pure code cleanup. Should I try to find a feature to add? Currently, I am doing all the easy stuff, and making it pep8 compliant and work through linter. I am also trying to delete some of the commented out code.

> I looked for and did not fine test/test_turtle. Did I miss something?
> Turtledemo is a partial substitute, but it might not exercise all 
> turtle functions.

I actually don't see the Turtle tests as well. Should creating unit tests for turtle.py be a separate ticket?

I am currently developing off of 3.4. I will try to run my patch through all the versions of Python 3 to make sure it is a clean fix.
History
Date User Action Args
2014-05-30 20:46:51Lita.Chosetrecipients: + Lita.Cho, terry.reedy, gregorlingl, jesstess
2014-05-30 20:46:51Lita.Chosetmessageid: <1401482811.58.0.970449085174.issue21573@psf.upfronthosting.co.za>
2014-05-30 20:46:51Lita.Cholinkissue21573 messages
2014-05-30 20:46:49Lita.Chocreate