On Monday 10 September 2007, Christopher Lenz wrote: > 3) Added bloat > There are a number of features currently in trunk that I think should > have never made there way in. Ticket cloning is one example, and I > know that many others share the sentiment that this shouldn't be in > the core. I don't care whether there's no appropriate plugin API for > implementing it outside of core; instead of just adding it, come up > with the extension API! We don't even have ticket deletion in core, > but we're adding cloning?!?
I have a ticket clone plugin (that works differently from what's in core). I'll try to push that out to trac-hacks. > There's also things like the "coloring" of directory listings, which > I would've vetoed, if (a) we had real policies on vetoing, and (b) I > hadn't been on my honeymoon at the time. I don't really understand > how anyone thinks this coloring by age is needed, but if there was a > need, add a friggin extension API instead! Let's call it > "IDirectoryListingAnnotator" or something. Hm. I like the coloring, personally. "Need"? Well, I'd say that 'file annotate' is a needed feature for the source browser. The coloring of the directory listing is the same thing, but at a file/directory granularity instead of per-line. > 4) Not enough testing > The unit test suite has seen many prolonged phases of being broken. A > lot of functionality has been added, but there are relatively few new > tests. This really needs to change. Now I know Eli is working on > functional testing on a branch, but that doesn't replace a good and > expanding unit test suite. Functional testing is on top of unit > tests, not a replacement. Agreed. I took the functional testing approach because I saw it as a way to get the broadest test coverage I could. Shallow testing, but broadly cover the user-visible features. Eli --~--~---------~--~----~------------~-------~--~----~ You received this message because you are subscribed to the Google Groups "Trac Development" 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/trac-dev?hl=en -~----------~----~----~----~------~----~------~--~---
