-
Notifications
You must be signed in to change notification settings - Fork 8
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
Development Checklist #50
Comments
@ariel-phet I would like to confirm the ordering of initial Dev QA testing and initial code review. Do you want to do dev testing first or code review first? UPDATE: and more generally, would you please review the checklist in this issue description and revise the order if anything is amiss? |
@samreid yes, dev testing should occur before code review. Dev testing may well reveal issues that require refactoring/changes. We like to ideally review code that is in a stable state, reasonably bug free, and unlikely to undergo many changes except for those raised by the review. |
I published http://www.colorado.edu/physics/phet/dev/html/circuit-construction-kit-dc/1.0.0-dev.160/circuit-construction-kit-dc_en.html and will test it on a few browsers before giving to the QA team. |
Since we are on dev testing and hopefully going into RC soon, this issue has served its usefulness, closing. |
The text was updated successfully, but these errors were encountered: