Re: [pgAdmin4][Patch] - SonarQube Issues - 12 (String literals should not be duplicated)

Поиск
Список
Период
Сортировка
От Akshay Joshi
Тема Re: [pgAdmin4][Patch] - SonarQube Issues - 12 (String literals should not be duplicated)
Дата
Msg-id CANxoLDdDrg5bZ0qgfyUgt_dbkkYqzp+vPzFFCP7=fetCvwg0eg@mail.gmail.com
обсуждение исходный текст
Ответ на [pgAdmin4][Patch] - SonarQube Issues - 12 (String literals should not be duplicated)  (Yogesh Mahajan <yogesh.mahajan@enterprisedb.com>)
Ответы Re: [pgAdmin4][Patch] - SonarQube Issues - 12 (String literals should not be duplicated)
Список pgadmin-hackers
Hi Yogesh

Instead of declaring global variables, I would suggest declaring them as a class variable (constants). Make the changes and resend the patch

On Mon, Aug 3, 2020 at 10:40 AM Yogesh Mahajan <yogesh.mahajan@enterprisedb.com> wrote:
Hi,

Please find patch which fixes 15+ issues related to rule - 
1.String literals should not be duplicated

Thanks,
Yogesh Mahajan
QA - Team
EnterpriseDB Corporation

Phone: +91-9741705709


--
Thanks & Regards
Akshay Joshi
pgAdmin Hacker | Sr. Software Architect
EDB Postgres
Mobile: +91 976-788-8246

В списке pgadmin-hackers по дате отправления:

Предыдущее
От: Aditya Toshniwal
Дата:
Сообщение: [pgAdmin][SonarQube] Fixes for 13 rules.
Следующее
От: Akshay Joshi
Дата:
Сообщение: pgAdmin 4 commit: Ensure that the original file format should be retain