On Sat, Jul 14, 2007 at 06:07:06PM +1200, Michael Koziarski wrote: > > I understand why patches get triaged, and > > there is the possibility of bad patches hanging around forever, but surely > > the policy of expiring old bug reports could also apply to half-baked > > patches, rather than instant closure? > > It's not just the possibility of patches sitting around, we had > hundreds of patches just sitting around and the submitters commented > that the lack of feedback was more frustrating than having a patch > 'rejected'.
Yes, lack of feedback sucks, but there are forms of feedback other than "closed defect: incomplete". > What was it about the closure that made it irksome? Tickets get > reopened all the time, closure isn't a death sentence. 'Closed' is one of the fundamental states of a ticket, and it means something fairly definite to me -- something along the lines of "absent some exceptional condition, this is done with". The thing that really got to me was that, if I had just reported the bug and left off the patch, it would have stayed open for someone else to deal with. However, having taken the extra time to hunt down a solution and post the patch (however incomplete it may have been), the ticket was closed and, absent further interaction from me, it was as though the defect was never reported. If the patch is dodgy, then reject the *patch*, not the entire defect report. I very nearly opened a completely separate bug report without a patch just to see what would happen. What would happen if a bug was reported by one person but an incomplete patch contributed by someone else? Would the report get closed (which to the initial reporter suggests the issue is solved) because the patch is dodgy? - Matt --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Ruby on Rails: Core" group. To post to this group, send email to [email protected] To unsubscribe from this group, send email to [EMAIL PROTECTED] For more options, visit this group at http://groups.google.com/group/rubyonrails-core?hl=en -~----------~----~----~----~------~----~------~--~---
