Postgres Pro
Downloads
Home   >   mailing lists

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

From Akshay Joshi
Subject Re: [pgAdmin4][Patch] - SonarQube Issues - 12 (String literals should not be duplicated)
Date
Msg-id CANxoLDdDrg5bZ0qgfyUgt_dbkkYqzp+vPzFFCP7=fetCvwg0eg@mail.gmail.com
Whole thread Raw
In response to [pgAdmin4][Patch] - SonarQube Issues - 12 (String literals should not be duplicated)  (Yogesh Mahajan <yogesh.mahajan@enterprisedb.com>)
Responses Re: [pgAdmin4][Patch] - SonarQube Issues - 12 (String literals should not be duplicated)  (Yogesh Mahajan <yogesh.mahajan@enterprisedb.com>)
List pgadmin-hackers
Tree view
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 by date:

Previous
From: Aditya Toshniwal
Date:
Subject: [pgAdmin][SonarQube] Fixes for 13 rules.
Next
From: Akshay Joshi
Date:
Subject: pgAdmin 4 commit: Ensure that the original file format should be retain