[darcs-users] darcs patch: refactor of SelectChanges (and 1 more)

Jason Dagit dagit at codersbase.com
Mon Oct 6 18:37:31 UTC 2008


I've only glanced over the patches, but I'll try to give some good feedback

Would you please be willing to send in 2 more patches on top of these to
address the following:
1) Please haddock more of your newly added functions.
2) Could you add some tests against your new UI?

Is the refactor motivated purely by the --lazy-deps option?  More
explanation of what you're doing here would be nice.  Sometimes it's hard to
infer your intentions from the code.


On Mon, Oct 6, 2008 at 11:20 AM, Florent Becker <florent.becker at ens-lyon.org
> wrote:

> Mon Oct  6 19:13:49 CEST 2008  Florent Becker <florent.becker at ens-lyon.org
> >
>  * refactor of SelectChanges
> Mon Oct  6 20:12:56 CEST 2008  Florent Becker <florent.becker at ens-lyon.org
> >
>  * added a --lazy-deps option
>  This option means that you don't get asked about patches which do not
> match
>  a --match, but are depended upon by a patch that does. These patches will
> get
>  silently selected.
> _______________________________________________
> darcs-users mailing list
> darcs-users at darcs.net
> http://lists.osuosl.org/mailman/listinfo/darcs-users
-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.osuosl.org/pipermail/darcs-users/attachments/20081006/905b920c/attachment.htm 

More information about the darcs-users mailing list