From: | Yogesh Mahajan <yogesh(dot)mahajan(at)enterprisedb(dot)com> |
---|---|
To: | pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org> |
Subject: | Re: Sonarqube fixes v2 |
Date: | 2022-09-09 13:53:31 |
Message-ID: | CAMa=N=Niv85gB5j_J1Cbka9zVBL35Rik1ro+0B9+M3k91bqS1Q@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | 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(dot)joshi(at)enterprisedb(dot)com>
wrote:
> Thanks, the patch applied
>
> On Tue, Aug 30, 2022 at 1:07 PM Yogesh Mahajan <
> yogesh(dot)mahajan(at)enterprisedb(dot)com> wrote:
>
>> Hello Team,
>>
>> Please find the attached patch which fixes around 25 sonarqube issues.
>>
>> Thanks,
>> Yogesh Mahajan
>> EnterpriseDB
>>
>
>
> --
>
> <http://www.enterprisedb.com>
>
> Akshay Joshi
>
> Principal Software Architect
>
> +91 9767888246
>
> www.enterprisedb.com
>
> <https://www.linkedin.com/company/edbpostgres>
> <https://twitter.com/edbpostgres?lang=en>
> <https://www.facebook.com/EDBpostgres>
> <https://www.instagram.com/EDBpostgres/>
>
Attachment | Content-Type | Size |
---|---|---|
Sonarqube_fixes_v2.patch | application/octet-stream | 12.2 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Khushboo Vashi | 2022-09-09 15:20:48 | Re: [pgAdmin4][Patch]: Remove Backgrid & Backform |
Previous Message | Pravesh Sharma | 2022-09-09 13:15:44 | Sonarqube fixes |