Re: Sonarqube fixes v2 - Mailing list pgadmin-hackers

From Akshay Joshi
Subject Re: Sonarqube fixes v2
Date
Msg-id CANxoLDcnW70Ntx8i4nRnw1xJO2VZqgqfMaC8x98xaJ1yqJicag@mail.gmail.com
Whole thread Raw
In response to Re: Sonarqube fixes v2  (Yogesh Mahajan <yogesh.mahajan@enterprisedb.com>)
List pgadmin-hackers
Thanks, the patch was applied.

On Fri, Sep 9, 2022 at 7:24 PM Yogesh Mahajan <yogesh.mahajan@enterprisedb.com> wrote:
Hi,

Please find the attached patch which fixes issue related to rule - 

1.Use concise character class syntax
2.Add a "break" statement or remove this "else" clause.
3.Replace this generic exception class with a more specific one.
4.Use a regular expression literal instead of the 'RegExp' constructor.
5.Use the opposite operator ("not in") instead.

Thanks,
Yogesh Mahajan
EnterpriseDB


On Tue, Aug 30, 2022 at 2:23 PM Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:
Thanks, the patch applied

On Tue, Aug 30, 2022 at 1:07 PM Yogesh Mahajan <yogesh.mahajan@enterprisedb.com> wrote:
Hello Team,

Please find the attached patch which fixes around 25 sonarqube issues.

Thanks,
Yogesh Mahajan
EnterpriseDB


--

Akshay Joshi

Principal Software Architect

+91 9767888246

www.enterprisedb.com

     



--

Akshay Joshi

Principal Software Architect

+91 9767888246

www.enterprisedb.com

     

pgadmin-hackers by date:

Previous
From: Akshay Joshi
Date:
Subject: Re: Sonarqube fixes
Next
From: Akshay Joshi
Date:
Subject: Re: [pgAdmin][patch]: sonarQube bug fixes