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

2018-02-19 Thread Dave Page
Thanks, patch applied. On Thu, Feb 15, 2018 at 4:15 PM, Murtuza Zabuawala < murtuza.zabuaw...@enterprisedb.com> wrote: > Thank you for reviewing Joao. > > > On Thu, Feb 15, 2018 at 8:37 PM, Joao De Almeida Pereira < > jdealmeidapere...@pivotal.io> wrote: > >> Hello Murtuza, >> I just run this

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

2018-02-15 Thread Murtuza Zabuawala
Thank you for reviewing Joao. On Thu, Feb 15, 2018 at 8:37 PM, Joao De Almeida Pereira < jdealmeidapere...@pivotal.io> wrote: > Hello Murtuza, > I just run this patch as you suggested(no errors reported) and also run it > through our CI pipeline and it look good > > Thanks > Joao > > On Wed,

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

2018-02-14 Thread Murtuza Zabuawala
Hi, PFA patch to fix the PEP-8 issues, To verify changes goto '../web' directory and run, pycodestyle --config=.pycodestyle ./regression/ pycodestyle --config=.pycodestyle ./pgadmin/setup/ pycodestyle --config=.pycodestyle ./pgadmin/settings/ pycodestyle --config=.pycodestyle