cordova-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Brian LeRoux...@brian.io>
Subject Re: Coho Clean-up
Date Fri, 25 Apr 2014 16:57:41 GMT
/me groans

(that co business is really elegant…a rare sight for js)


On Thu, Apr 24, 2014 at 9:56 PM, Michal Mocny <mmocny@chromium.org> wrote:

> 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