[openstack-dev] [solum] pep8 - splitting expressions

Kevin L. Mitchell kevin.mitchell at rackspace.com
Fri Sep 12 17:23:56 UTC 2014


On Thu, 2014-09-11 at 18:35 -0500, Ed Leafe wrote:
> On Sep 11, 2014, at 5:05 PM, Kevin L. Mitchell <kevin.mitchell at 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.mitchell at rackspace.com>
Rackspace




More information about the OpenStack-dev mailing list