[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v4 2/3] backup: Adds Backup Tool
From: |
John Snow |
Subject: |
Re: [Qemu-devel] [PATCH v4 2/3] backup: Adds Backup Tool |
Date: |
Wed, 13 Sep 2017 16:04:23 -0400 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.3.0 |
On 09/11/2017 01:02 AM, Fam Zheng wrote:
> On Fri, 09/08 22:11, Ishani Chugh wrote:
>> +def build_parser():
>> + backup_tool = BackupTool()
>> + parser = ArgumentParser()
>> + subparsers = parser.add_subparsers(title='Subcommands',
>> + description='Valid Subcommands',
>> + help='Subcommand help')
>> + guest_parser = subparsers.add_parser('guest', help='Manage guest(s)')
>> + guest_subparsers = guest_parser.add_subparsers(title='Guest Subparser')
>> +# Guest list
>
> Odd indentation of comments. Please align the # at 4th colume line other code
> lines. The same to below.
>
Sorry, that's probably my fault based on how I suggested the comments in
the previous email review. (:
Fam is right though, the pythonic way to comment is by aligning the
octothorpe to column 4 instead of leaving it at column 0.
--js