[Sugar-devel] [PATCH 10/21 sugar-toolkit] style cleanup: use """ everywhere

Simon Schampijer simon at schampijer.de
Fri Oct 29 06:53:47 EDT 2010


Reviewed-by: Simon Schampijer simon at laptop.org

Yes, we should set on one and mark this in our coding style document. 
Btw, we should update that as well.

On 10/15/2010 11:01 PM, Sascha Silbe wrote:
> Most of the code uses """, so adjust the few deviations.
>
> Signed-off-by: Sascha Silbe<sascha-pgp at silbe.org>
>
> diff --git a/src/sugar/activity/bundlebuilder.py b/src/sugar/activity/bundlebuilder.py
> index 7bf02ed..d058b55 100644
> --- a/src/sugar/activity/bundlebuilder.py
> +++ b/src/sugar/activity/bundlebuilder.py
> @@ -277,7 +277,7 @@ class Installer(object):
>
>
>   def cmd_dev(config, args):
> -    '''Setup for development'''
> +    """Setup for development"""
>
>       if args:
>           print 'Usage: %prog dev'
> @@ -297,7 +297,7 @@ def cmd_dev(config, args):
>
>
>   def cmd_dist_xo(config, args):
> -    '''Create a xo bundle package'''
> +    """Create a xo bundle package"""
>
>       if args:
>           print 'Usage: %prog dist_xo'
> @@ -308,7 +308,7 @@ def cmd_dist_xo(config, args):
>
>
>   def cmd_fix_manifest(config, args):
> -    '''Add missing files to the manifest'''
> +    """Add missing files to the manifest"""
>
>       if args:
>           print 'Usage: %prog fix_manifest'
> @@ -319,7 +319,7 @@ def cmd_fix_manifest(config, args):
>
>
>   def cmd_dist_source(config, args):
> -    '''Create a tar source package'''
> +    """Create a tar source package"""
>
>       if args:
>           print 'Usage: %prog dist_source'
> @@ -330,7 +330,7 @@ def cmd_dist_source(config, args):
>
>
>   def cmd_install(config, args):
> -    '''Install the activity in the system'''
> +    """Install the activity in the system"""
>
>       parser = OptionParser(usage='usage: %prog install [options]')
>       parser.add_option('--prefix', dest='prefix', default=sys.prefix,
> @@ -345,7 +345,7 @@ def cmd_install(config, args):
>
>
>   def cmd_genpot(config, args):
> -    '''Generate the gettext pot file'''
> +    """Generate the gettext pot file"""
>
>       if args:
>           print 'Usage: %prog genpot'
> @@ -384,7 +384,7 @@ def cmd_genpot(config, args):
>
>
>   def cmd_build(config, args):
> -    '''Build generated files'''
> +    """Build generated files"""
>
>       if args:
>           print 'Usage: %prog build'
> diff --git a/src/sugar/bundle/activitybundle.py b/src/sugar/bundle/activitybundle.py
> index dfec7a8..18a08bc 100644
> --- a/src/sugar/bundle/activitybundle.py
> +++ b/src/sugar/bundle/activitybundle.py
> @@ -346,8 +346,8 @@ class ActivityBundle(Bundle):
>           return install_path
>
>       def install_mime_type(self, install_path):
> -        ''' Update the mime type database and install the mime type icon
> -        '''
> +        """ Update the mime type database and install the mime type icon
> +        """
>           xdg_data_home = os.getenv('XDG_DATA_HOME',
>                                     os.path.expanduser('~/.local/share'))
>
> --
> 1.7.1
>
> _______________________________________________
> Sugar-devel mailing list
> Sugar-devel at lists.sugarlabs.org
> http://lists.sugarlabs.org/listinfo/sugar-devel
>



More information about the Sugar-devel mailing list