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 <[email protected]>
An: "Pharo Development List" <[email protected]>
Betreff: [Pharo-dev] comments :)

Thanks for all these comments!
On 05 Feb 2014, at 00:27, GitHub <[email protected]> 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]




Reply via email to