Thanks for reviewing Joao.Yeah missed to run tests. I assumed its just a SQL formatting it wouldn't hurt anything but I was wrong :)Here's updated patch.--Regards,Murtuza ZabuawalaEnterpriseDB: http://www.enterprisedb.comThe Enterprise PostgreSQL CompanyOn Tue, Apr 10, 2018 at 8:18 PM, Joao De Almeida Pereira <jdealmeidapereira@pivotal.io> wrote:Hi Murtuza,Looks like the tests are failing due to this change.I also got some strange result on the linter, but I believe it is something related to this machine.ThanksJoaoOn Tue, Apr 10, 2018 at 6:20 AM Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote:Hi Dave,A minor fix for explain functionality.Issue: Due to multiline explain options in the SQL template CodeMirror was failing to highlight the exact line of error.--Regards,Murtuza ZabuawalaEnterpriseDB: http://www.enterprisedb.comThe Enterprise PostgreSQL Company
Hi Murtuza,Looks like the tests are failing due to this change.I also got some strange result on the linter, but I believe it is something related to this machine.ThanksJoaoOn Tue, Apr 10, 2018 at 6:20 AM Murtuza Zabuawala <murtuza.zabuawala@enterprisedb.com> wrote:Hi Dave,A minor fix for explain functionality.Issue: Due to multiline explain options in the SQL template CodeMirror was failing to highlight the exact line of error.--Regards,Murtuza ZabuawalaEnterpriseDB: http://www.enterprisedb.comThe Enterprise PostgreSQL Company
Hi Dave,A minor fix for explain functionality.Issue: Due to multiline explain options in the SQL template CodeMirror was failing to highlight the exact line of error.--Regards,Murtuza ZabuawalaEnterpriseDB: http://www.enterprisedb.comThe Enterprise PostgreSQL Company
pgadmin-hackers by date:
Соглашаюсь с условиями обработки персональных данных