[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