Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
1 task
zepumph opened this issue Sep 15, 2023 · 1 comment
Open
1 task

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

zepumph opened this issue Sep 15, 2023 · 1 comment

Comments

@zepumph
Copy link
Member

zepumph commented Sep 15, 2023

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)

UPDATE by @samreid, currently at:

  • 315 occurrences of this issue in scenery/
@pixelzoom
Copy link
Contributor

pixelzoom commented Sep 15, 2023

Also adding... Additional TODOs should not be created that point to this issue. And there are currently 325 TODOs that point to this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants