On Mon, Jan 8, 2018 at 8:52 PM, Dave Page <dpage@pgadmin.org> wrote:HiOn Mon, Jan 8, 2018 at 3:18 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:On Mon, Jan 8, 2018 at 8:31 PM, Dave Page <dpage@pgadmin.org> wrote:HiI think these are impossibly big to properly review by hand, especially where much of them are whitespace changes.I ran both jasmine test framework, and feature tests.Both are running fine. I think the best option is to check the regression tests all pass, do some manual testing yourself, and then commit. Done some manual testing.It would be helpful, If somebody can help with some more testing, which I may have missed.Please ask one of the team (other than me) :-):-)-- Thanks, Ashesh -- Thanks, Ashesh On Mon, Jan 8, 2018 at 1:55 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:Hi Dave/Team,I found many problems while 'eslint' utility on all the static javascript files of pgAdmin 4.i.e. <pgAdmin4_path>/web/node_module/.bin/eslint <file_name>I had fixed all errors reported by eslint for those files.I have created six patches for ease of maintenance, as possibilities of conflicting with other patches.* Browser nodes javascript files* Browser specific javascript files* pgAdmin common javascript files* SQLEditor/DataGrid javascript files* Tools javascript files* Miscellaneous modules javascript filesAll patches are independent of each other.Please review it, and share your opinion. --Thanks & Regards,Ashesh VashiEnterpriseDB INDIA: Enterprise PostgreSQL Companyhttp://www.linkedin.com/in/asheshvashi-- Dave PageBlog: http://pgsnake.blogspot.comTwitter: @pgsnakeEnterpriseDB UK: http://www.enterprisedb.comThe Enterprise PostgreSQL Company -- Dave PageBlog: http://pgsnake.blogspot.comTwitter: @pgsnakeEnterpriseDB UK: http://www.enterprisedb.comThe Enterprise PostgreSQL Company
HiOn Mon, Jan 8, 2018 at 3:18 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:On Mon, Jan 8, 2018 at 8:31 PM, Dave Page <dpage@pgadmin.org> wrote:HiI think these are impossibly big to properly review by hand, especially where much of them are whitespace changes.I ran both jasmine test framework, and feature tests.Both are running fine. I think the best option is to check the regression tests all pass, do some manual testing yourself, and then commit. Done some manual testing.It would be helpful, If somebody can help with some more testing, which I may have missed.Please ask one of the team (other than me) :-)
On Mon, Jan 8, 2018 at 8:31 PM, Dave Page <dpage@pgadmin.org> wrote:HiI think these are impossibly big to properly review by hand, especially where much of them are whitespace changes.I ran both jasmine test framework, and feature tests.Both are running fine. I think the best option is to check the regression tests all pass, do some manual testing yourself, and then commit. Done some manual testing.It would be helpful, If somebody can help with some more testing, which I may have missed.
HiI think these are impossibly big to properly review by hand, especially where much of them are whitespace changes.
I think the best option is to check the regression tests all pass, do some manual testing yourself, and then commit.
-- Thanks, Ashesh On Mon, Jan 8, 2018 at 1:55 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:Hi Dave/Team,I found many problems while 'eslint' utility on all the static javascript files of pgAdmin 4.i.e. <pgAdmin4_path>/web/node_module/.bin/eslint <file_name>I had fixed all errors reported by eslint for those files.I have created six patches for ease of maintenance, as possibilities of conflicting with other patches.* Browser nodes javascript files* Browser specific javascript files* pgAdmin common javascript files* SQLEditor/DataGrid javascript files* Tools javascript files* Miscellaneous modules javascript filesAll patches are independent of each other.Please review it, and share your opinion. --Thanks & Regards,Ashesh VashiEnterpriseDB INDIA: Enterprise PostgreSQL Companyhttp://www.linkedin.com/in/asheshvashi-- Dave PageBlog: http://pgsnake.blogspot.comTwitter: @pgsnakeEnterpriseDB UK: http://www.enterprisedb.comThe Enterprise PostgreSQL Company -- Dave PageBlog: http://pgsnake.blogspot.comTwitter: @pgsnakeEnterpriseDB UK: http://www.enterprisedb.comThe Enterprise PostgreSQL Company
-- Thanks, Ashesh On Mon, Jan 8, 2018 at 1:55 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:Hi Dave/Team,I found many problems while 'eslint' utility on all the static javascript files of pgAdmin 4.i.e. <pgAdmin4_path>/web/node_module/.bin/eslint <file_name>I had fixed all errors reported by eslint for those files.I have created six patches for ease of maintenance, as possibilities of conflicting with other patches.* Browser nodes javascript files* Browser specific javascript files* pgAdmin common javascript files* SQLEditor/DataGrid javascript files* Tools javascript files* Miscellaneous modules javascript filesAll patches are independent of each other.Please review it, and share your opinion. --Thanks & Regards,Ashesh VashiEnterpriseDB INDIA: Enterprise PostgreSQL Companyhttp://www.linkedin.com/in/asheshvashi-- Dave PageBlog: http://pgsnake.blogspot.comTwitter: @pgsnakeEnterpriseDB UK: http://www.enterprisedb.comThe Enterprise PostgreSQL Company
On Mon, Jan 8, 2018 at 1:55 PM, Ashesh Vashi <ashesh.vashi@enterprisedb.com> wrote:Hi Dave/Team,I found many problems while 'eslint' utility on all the static javascript files of pgAdmin 4.i.e. <pgAdmin4_path>/web/node_module/.bin/eslint <file_name>I had fixed all errors reported by eslint for those files.I have created six patches for ease of maintenance, as possibilities of conflicting with other patches.* Browser nodes javascript files* Browser specific javascript files* pgAdmin common javascript files* SQLEditor/DataGrid javascript files* Tools javascript files* Miscellaneous modules javascript filesAll patches are independent of each other.Please review it, and share your opinion. --Thanks & Regards,Ashesh VashiEnterpriseDB INDIA: Enterprise PostgreSQL Companyhttp://www.linkedin.com/in/asheshvashi-- Dave PageBlog: http://pgsnake.blogspot.comTwitter: @pgsnakeEnterpriseDB UK: http://www.enterprisedb.comThe Enterprise PostgreSQL Company
Hi Dave/Team,I found many problems while 'eslint' utility on all the static javascript files of pgAdmin 4.i.e. <pgAdmin4_path>/web/node_module/.bin/eslint <file_name>I had fixed all errors reported by eslint for those files.I have created six patches for ease of maintenance, as possibilities of conflicting with other patches.* Browser nodes javascript files* Browser specific javascript files* pgAdmin common javascript files* SQLEditor/DataGrid javascript files* Tools javascript files* Miscellaneous modules javascript filesAll patches are independent of each other.Please review it, and share your opinion. --Thanks & Regards,Ashesh VashiEnterpriseDB INDIA: Enterprise PostgreSQL Companyhttp://www.linkedin.com/in/asheshvashi
--
Thanks & Regards,Ashesh VashiEnterpriseDB INDIA: Enterprise PostgreSQL Company
http://www.linkedin.com/in/asheshvashi
pgadmin-hackers by date:
Соглашаюсь с условиями обработки персональных данных