fixed and also fixed (and verified) the simpler pattern matching you
recommended. PTAL.


On Thu, Apr 3, 2014 at 1:49 PM, brisbink <notifications@github.com> wrote:

> In demo-suite/demos/image-magick/main.py:
>
> >        instances.append(gce.Instance(
> > -          name='%s-%d' % (DEMO_NAME, i),
> > -          image_project_id=image_project,
> > -          image_name=image_name,
> > +          name=instance_name, network_interfaces=ext_net,
>
> OK, I'm going to be a bit nit-picky here. The more I look at this line,
> the more I think "network_interfaces=ext_net," should probably go on the
> next line. Thanks!
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/GoogleCloudPlatform/compute-appengine-demo-suite-python/pull/18/files#r11272006>
> .
>



-- 
Marc Cohen
marccohen@google.com <mco@google.com>
http://about.me/marc1