public inbox for [email protected]  
help / color / mirror / Atom feed
From: Murtuza Zabuawala <[email protected]>
To: Joao De Almeida Pereira <[email protected]>
Cc: pgadmin-hackers <[email protected]>
Subject: Re: [pgAdmin4][RM#3175] Fix PEP-8 issues
Date: Mon, 5 Mar 2018 21:53:56 +0530
Message-ID: <CAKKotZRb8UJkbMui+Zd1K8cp2J-KoGEvkpFbC1WpFRWfhS9j=w@mail.gmail.com> (raw)
In-Reply-To: <CAE+jjanM6exPkvYs+ECg4n1CD55Gs8_TiMzGycacvoTcnkSpJw@mail.gmail.com>
References: <CAKKotZQQecSw7Og3yXn8rK9yY_qKCmhb-Dj4Q2t9GDyRc9_D_w@mail.gmail.com>
	<CAE+jjanM6exPkvYs+ECg4n1CD55Gs8_TiMzGycacvoTcnkSpJw@mail.gmail.com>

Thanks Joao for reviewing.


On Mon, Mar 5, 2018 at 8:58 PM, Joao De Almeida Pereira <
[email protected]> wrote:

> Hi Murtuza,
>
> Looks good, and all tests are green. I run the commands provided and
> nothing pops up.
>
> Only missing 374 :D
>
​:-) ​


>
> Thanks
> Joao
>
> On Mon, Mar 5, 2018 at 7:09 AM Murtuza Zabuawala <murtuza.zabuawala@
> enterprisedb.com> wrote:
>
>> Hi,
>>
>> PFA patch to fix PEP8 issues in Browser & Database modules,
>> pycodestyle --config=.pycodestyle pgadmin/browser/__init__.py
>> pycodestyle --config=.pycodestyle pgadmin/browser/tests/
>> pycodestyle --config=.pycodestyle pgadmin/browser/server_groups/
>> servers/databases/__init__.py
>> pycodestyle --config=.pycodestyle pgadmin/browser/server_groups/
>> servers/databases/tests/
>>
>>
>>
>> --
>> Regards,
>> Murtuza Zabuawala
>> EnterpriseDB: http://www.enterprisedb.com
>> The Enterprise PostgreSQL Company
>>
>>


view thread (8+ messages)  latest in thread

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: [email protected]
  Cc: [email protected], [email protected]
  Subject: Re: [pgAdmin4][RM#3175] Fix PEP-8 issues
  In-Reply-To: <CAKKotZRb8UJkbMui+Zd1K8cp2J-KoGEvkpFbC1WpFRWfhS9j=w@mail.gmail.com>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by agora; see mirroring instructions
for how to clone and mirror all data and code used for this inbox