Re: Sonarqube fixes v2 - Mailing list pgadmin-hackers

From Yogesh Mahajan
Subject Re: Sonarqube fixes v2
Date
Msg-id CAMa=N=Niv85gB5j_J1Cbka9zVBL35Rik1ro+0B9+M3k91bqS1Q@mail.gmail.com
Whole thread Raw
In response to Re: Sonarqube fixes  (Akshay Joshi <akshay.joshi@enterprisedb.com>)
Responses Re: Sonarqube fixes v2  (Akshay Joshi <akshay.joshi@enterprisedb.com>)
List pgadmin-hackers
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

     

Attachment

pgadmin-hackers by date:

Previous
From: Pravesh Sharma
Date:
Subject: Sonarqube fixes
Next
From: Khushboo Vashi
Date:
Subject: Re: [pgAdmin4][Patch]: Remove Backgrid & Backform