On Mon, 2010-11-15 at 06:54 -0500, Martyn Taylor wrote:
After to speaking to some of the guys on the dev team it seems some people have differences of opinion with regards to some of the ways that things are done or are maybe unsure on the best way to do something.
I suggest we write some basic guidelines for any topics that people are unsure of, or if they're seeing things that they feel should be done differently.
I think a good start would be to get any suggestions from people who feel that things should be done differently and a list of things people are unsure of. Then we could maybe collate all the stuff together and stick it in the wiki - if its a standard practice or common knowledge we could maybe add a link to a web page that explains.
Some things that we've already discussed include:
Testing: lmartinc
Error Handling: mtaylor When to use errors, throw/catch exceptions
Patch Submissions: mtaylor I feel that all refactoring should be added in separate patches, rather than including this in any bug fixes or tasks. This way we can see exactly what the changes are wft particular bugs etc...
Thanks guys
Martyn
Martyn, this is a good idea. I hope everyone feels comfortable enough to ask questions, both here and in irc. We are an open group, so questions/debate are always welcome. Maybe once we have assembled said list of questions on the wiki, and built out decent answers, we can promote it to regular site content to make it more 'official'.
FWIW, I completely agree that refactors should be done in a separate patch, trying to squeeze them into a bugfix not only is harder to read for changes, but increases the chances of inadvertently breaking something else in the process.
AFA the testing and error-handling bits above, are those questions, or a promise to write something up? If the former, what are the questions?
-j