On Tue, Jul 02, 2013 at 11:29:16AM +0200, Gwenaël Casaccio wrote:
> Hi,
> 
> I've extended gst-sunit to run sunit with a package description file
> thanks to the Holger patch.

look's great

>           | pkg |
> -         pkg := PackageLoader packageAt: arg.
> +            (arg endsWith: '.xml') 
> +                ifTrue: [ pkg := PackageLoader loadPackageFromFile: 
> Directory working / arg ]
> +                ifFalse: [ pkg := PackageLoader packageAt: arg ].

mixing tabs/spaces? I think paolo pointed me to a book of kent beck that
argued for

   pkg := (arg endsWith: '.xml')
           ifTrue: [PackageLoader ...]
           ifFalse: [PackageLoader ...]

if you agree i will make these changes and apply.

_______________________________________________
help-smalltalk mailing list
help-smalltalk@gnu.org
https://lists.gnu.org/mailman/listinfo/help-smalltalk

Reply via email to