search

phetsims / kite

A library for creating, manipulating and displaying 2D shapes in JavaScript.
MIT License
13 stars 6 forks source link

TODOs in this repo should point to useful github issues #102

Closed zepumph closed 11 months ago

zepumph commented 1 year ago

PhET has a policy that TODOs in production code are tied to a github issue to help in scheduling their work. This policy was created after many TODOs were created without a corresponding github issue associated with them in this repo. Never the less, this code is still production code, and TODOs should adhere to this philosophy. I'll mark TODOs in this repo to this issue now, but this should NOT be a catch all, work should be done to handle, triage, schedule and fix the work outlined by the TODOs in this repo (it is production code!).

(see work done in phetsims/tasks#1129 also)

marlitas commented 11 months ago

this is a duplicate of https://github.com/phetsims/kite/issues/76.

Although it now seems like there are 94 instead of 70. Closing as a duplicate.

zepumph commented 11 months ago

Updated todos pointing to this issue so it doesn't reopen tonight.

marlitas commented 11 months ago

Ope thanks for that. I believe I did a couple of others that I should hit as well.

marlitas commented 11 months ago

You did this everywhere. What a gem. Thanks!