Re: [pgAdmin4][RM#3175] Fix PEP-8 issues

2018-03-07 Thread Dave Page
Thanks, applied! On Mon, Mar 5, 2018 at 4:23 PM, Murtuza Zabuawala < murtuza.zabuaw...@enterprisedb.com> wrote: > Thanks Joao for reviewing. > > > On Mon, Mar 5, 2018 at 8:58 PM, Joao De Almeida Pereira < > jdealmeidapere...@pivotal.io> wrote: > >> Hi Murtuza, >> >> Looks good, and all tests are

Re: [pgAdmin4][RM#3175] Fix PEP-8 issues

2018-03-05 Thread Murtuza Zabuawala
Thanks Joao for reviewing. On Mon, Mar 5, 2018 at 8:58 PM, Joao De Almeida Pereira < jdealmeidapere...@pivotal.io> 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

Re: [pgAdmin4][RM#3175] Fix PEP-8 issues

2018-03-05 Thread Joao De Almeida Pereira
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.zabuaw...@enterprisedb.com> wrote: > Hi, > > PFA patch to fix PEP8 issues in Browser & Database modules,

[pgAdmin4][RM#3175] Fix PEP-8 issues

2018-03-05 Thread Murtuza Zabuawala
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=