cordova-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Michal Mocny <mmo...@chromium.org>
Subject Re: Coho Clean-up
Date Fri, 25 Apr 2014 04:56:15 GMT
On Thu, Apr 24, 2014 at 9:42 PM, Andrew Grieve <agrieve@chromium.org> wrote:

> Yes, I will say that I was super impressed at how cleanly the code
> converted to co-routines.
>
> I had started down the road of promises and quickly determined it was
> infeasible.
>
> With "co", 99% of the change was converting function() -> function*(),
> and added "yield" in-front of anything async.
>
> There's even code like:
> if (cond1 && yield someAsyncThing()) {}
>
> Future is bright :)
>
You Promise?


>
>
> On Thu, Apr 24, 2014 at 8:48 PM, Brian LeRoux <b@brian.io> wrote:
> > ok, this looks awesome. also this is good fun:
> >
> > https://github.com/apache/cordova-coho/blob/master/package.json#L10
> >
> >
> > On Thu, Apr 24, 2014 at 3:46 PM, Steven Gill <stevengill97@gmail.com>
> wrote:
> >
> >> Thanks Andrew! Can't wait to check it out!
> >>
> >>
> >> On Thu, Apr 24, 2014 at 2:15 PM, Anis KADRI <anis.kadri@gmail.com>
> wrote:
> >>
> >> > cool
> >> >
> >> >
> >> > On Thu, Apr 24, 2014 at 1:50 PM, Shazron <shazron@gmail.com> wrote:
> >> >
> >> > > Thanks for this Andrew!
> >> > >
> >> > >
> >> > > On Thu, Apr 24, 2014 at 10:42 AM, Andrew Grieve <
> agrieve@chromium.org
> >> > > >wrote:
> >> > >
> >> > > > I went coho crazy last night & this morning. Made two significant
> >> > > changes:
> >> > > >
> >> > > > 1. It doesn't use shelljs.exec() anymore. Instead, it uses
> >> co-routines
> >> > > > and superspawn. This requires that your use node v0.11 or greater
> to
> >> > > > run it.
> >> > > >
> >> > > > 2. Broke all commands into their own files.
> >> > > >
> >> > > > Many commands now run faster due to no shell.exec(). The node
> update
> >> > > > is a paint, but meh. If you use nvm, it's just nvm install 0.11
&&
> >> nvm
> >> > > > alias default 0.11.
> >> > > >
> >> > >
> >> >
> >>
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message