This page will be used to collect comments about PDE and our development process that were noticed during 3.5. What did we do well? What could be better? What should we do next release?
Coding Best Practices
- If you write new core/model/non-UI code, write tests for it
- Don't release a major change just before a milestone release or even right before an I build
- If you create any kind of UI make sure it has a help topic. Also ensure that you create the help topic(s) at the time the UI is created. DO NOT leave it until the doc pass at the end of the release.
- If you add someone as a reviewer, add them as a cc
- Only mark a bug with a specific target milestone if you seriously plan on fixing it during that milestone
- If you are unsure about a change or the code you are committing is a major change, add a reviewer and get them to verify it
- In 3.5 the target platform changes ended up dropping too late M6/M7 to get proper feedback. We recognized this as a risk beforehand and still went forward and we did get burned for this to some extent. There were too many RC fixes required to get it working.
- Regularly run a profiler (or the like) over any new larger changes to catch performance regressions earlier
- It would be good to always have another pair of eyes (or two) verify all fixes
- We released 3.5 with two bugs marked BLOCKER and one CRITICAL. I haven't really checked if these really are severe defects, but we probably can have communicated better on the actual status of these issues.
- Planning for API tooling was better than for PDE UI in that plan items were more concrete and scoped. The PDE UI plan was more vague - we knew we wanted to make major improvements to target platforms (which we did), but the plan items were not as well described/scoped/understood (this is an another reason it arrived late in the release). Similarly, support for categories in exported p2 repos appeared late. In 3.6 we need better plan items/descriptions and understanding of the scope of work items. Big items must come first to avoid stability issues late in the cycle.
- PDE has a diverse committer pool. This is good. As the committers represent different interests from different organizations, we need to ensure that everyone participates in the planning process to understand what work will be going on and where we can benefit from common interests. As well, we need to ensure there is good ongoing communication throughout the cycle.
- When possible, doc was written during the cycle (for example, when API tooling problem severity settings were re-worked). This greatly reduced the burden at the end of the cycle. There was still plenty of target platform doc to be created at the end (but one could argue that this was in tandem with the feature delivery).
- Create better test plan pages well before milestone closes. So that, during test pass week, everyone have can focus on specific portions for testing instead of everyone tests everything. May be 2 set of eyes for each feature testing.
- Automated builds seemed less reliable than usual for this release - in part due to the many changes in PDE build as it evolved to leverage more p2 features.