Обсуждение: [pgAdmin4][Patch] - SonarQube Issues - 12 (String literals should not be duplicated)

Поиск
Список
Период
Сортировка

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

От
Yogesh Mahajan
Дата:
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
Вложения

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

От
Akshay Joshi
Дата:
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

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

От
Yogesh Mahajan
Дата:
Hello,

Please find the attached patch - 


Thanks,
Yogesh Mahajan
QA - Team
EnterpriseDB Corporation

Phone: +91-9741705709


On Mon, Aug 3, 2020 at 12:56 PM Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:
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

Вложения