Hi Ben if we would just like to increase the number of cases/issues using a "case per class" would have been better. Adding final quality improvements (like docu) is usual before releases not only in the ST world. For the reasons: A single person can only do it step by step and therefore I used the granularity of projects/packages. This also helps reintegration /mergin back into external managed parts. There was no reaction onto my call for action from others. So instead of grumbling afterwards better use your keyboard to comment on classes ;) Thx T.
Gesendet: Donnerstag, 06. Februar 2014 um 10:43 Uhr Von: btc@openinworld.com An: "Pharo Development List" <pharo-dev@lists.pharo.org> Betreff: Re: [Pharo-dev] comments :)
Good work. Minor thing though - could they maybe grouped under one Case. I'd hate for someone to look at the massive amount work done for Pharo 3 indicated by the cases close and think "Its just lots of comment updates" Sure it would end up being a long running case - with a lot of slices (each based off the most recent image). You'd have to see how that panned out, and itf it got too long, just link to a new case for the next group. Or maybe there is some difficulty that I can't forsee. cheers -ben
Torsten Bergmann wrote:
Still 1017 to go. Feel free to help:
Smalltalk allClasses select: [:each | each comment includesSubstring: 'xxxx' ]
External Package owners should also reintegrate too so these do not get lost when a new version of external packages get integrated. (for instance https://pharo.fogbugz.com/f/cases/12812 for Spec)
step by step it goes ...
Bye T.
Gesendet: Mittwoch, 05. Februar 2014 um 08:36 Uhr Von: Pharo4Stef <pharo4Stef@free.fr> An: "Pharo Development List" <pharo-dev@lists.pharo.org> Betreff: [Pharo-dev] comments :)
Thanks for all these comments!
On 05 Feb 2014, at 00:27, GitHub <noreply@github.com> wrote: 30742 12794 Class comments missing in UpdateStreamer package https://pharo.fogbugz.com/f/cases/12794[https://pharo.fogbugz.com/f/cases/12794]
12802 Class comments missing for Kernel package https://pharo.fogbugz.com/f/cases/12802[https://pharo.fogbugz.com/f/cases/12802]
12804 Class comment missing in AST packages https://pharo.fogbugz.com/f/cases/12804[https://pharo.fogbugz.com/f/cases/12804]
12797 Class comments missing for tools package https://pharo.fogbugz.com/f/cases/12797[https://pharo.fogbugz.com/f/cases/12797]
12796 Class comments missing for Traits package https://pharo.fogbugz.com/f/cases/12796[https://pharo.fogbugz.com/f/cases/12796]
12798 Class comments missing in GroupManager package https://pharo.fogbugz.com/f/cases/12798[https://pharo.fogbugz.com/f/cases/12798]
12806 Class comment missing in package FreeType + categorizing for FreeTypeCacheTest https://pharo.fogbugz.com/f/cases/12806[https://pharo.fogbugz.com/f/cases/12806]
12800 EyeContextInspector should be renamed to EyeMethodContextInspector https://pharo.fogbugz.com/f/cases/12800[https://pharo.fogbugz.com/f/cases/12800]