Re: Sonarqube fixes v2

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

In response to

Responses

Browse pgadmin-hackers by date

  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