On Thu, 2014-09-11 at 18:35 -0500, Ed Leafe wrote: > On Sep 11, 2014, at 5:05 PM, Kevin L. Mitchell <kevin.mitch...@rackspace.com> > wrote: > > > I'd suggest trying: > > > > res = amodel.Assemblies( > > uri=common.ASSEM_URI_STR % pecan.request.host_url, > > name='Solum_CAMP_assemblies', > > type='assemblies', > > description=common.ASSEM_DESC_STR, > > assembly_links=a_links, > > parameter_definitions_uri=common.ASSEM_PARAM_STR % > > pecan.request.host_url) > > > > By moving the first argument to a line by itself, pep8 can be satisfied > > by indenting the following lines by 4 spaces. > > When not using visual indentation, the standard is to indent two > levels (i.e., 8 spaces), to distinguish it from typical blocks. But > yes, I much prefer this than creating temporary names to accommodate > the visual indentation style.
Using an 8 space indent here would result in an E126 pep8 error, which is why I use 4 spaces for this case. Of course, I don't know if Solum enforces E126. -- Kevin L. Mitchell <kevin.mitch...@rackspace.com> Rackspace _______________________________________________ OpenStack-dev mailing list OpenStack-dev@lists.openstack.org http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev