Re: Patch for SonarQube code scan fixes.

From: Dave Page <dpage(at)pgadmin(dot)org>
To: Aditya Toshniwal <aditya(dot)toshniwal(at)enterprisedb(dot)com>
Cc: Nikhil Mohite <nikhil(dot)mohite(at)enterprisedb(dot)com>, pgadmin-hackers <pgadmin-hackers(at)postgresql(dot)org>
Subject: Re: Patch for SonarQube code scan fixes.
Date: 2020-09-08 10:23:08
Message-ID: CA+OCxozngZBtnsQywQJ+rnUp-scXmYmzCeFoxQ9oZtvY5MxW4g@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgadmin-hackers

On Tue, Sep 8, 2020 at 11:01 AM Aditya Toshniwal <
aditya(dot)toshniwal(at)enterprisedb(dot)com> wrote:

> Hi,
>
> sqlautocomplete code is mostly inherited from the pgcli project. Changing
> it might create trouble in updating the code with new versions.
>

Send a patch upstream :-)

>
> On Tue, Sep 8, 2020 at 2:59 PM Nikhil Mohite <
> nikhil(dot)mohite(at)enterprisedb(dot)com> wrote:
>
>> Hi Team,
>>
>> I have fixed some code smell issues in the SonarQube scan, PFA patch.
>> Details as follows:
>>
>> 1. psycopg2/connection:
>>
>> - Refactor this function to reduce its Cognitive Complexity from 32
>> to the 15 allowed.
>> - Refactor this function to reduce its Cognitive Complexity from 17
>> to the 15 allowed.
>>
>> 2. psycopg2/server_manager:
>>
>> - Refactor this function to reduce its Cognitive Complexity from 20
>> to the 15 allowed.
>> - Refactor this function to reduce its Cognitive Complexity from 33
>> to the 15 allowed.
>> - Refactor this function to reduce its Cognitive Complexity from 26
>> to the 15 allowed.
>>
>> 3. sqlautocomplete/parseutils:
>>
>> - Refactor this function to reduce its Cognitive Complexity from 23
>> to the 15 allowed.
>>
>>
>>
>> --
>> *Thanks & Regards,*
>> *Nikhil Mohite*
>> *Software Engineer.*
>> *EDB Postgres* <https://www.enterprisedb.com/>
>> *Mob.No: +91-7798364578.*
>>
>
>
> --
> Thanks,
> Aditya Toshniwal
> pgAdmin hacker | Sr. Software Engineer | *edbpostgres.com*
> <http://edbpostgres.com>
> "Don't Complain about Heat, Plant a TREE"
>

--
Dave Page
Blog: http://pgsnake.blogspot.com
Twitter: @pgsnake

EDB: http://www.enterprisedb.com

In response to

Responses

Browse pgadmin-hackers by date

  From Date Subject
Next Message Aditya Toshniwal 2020-09-08 10:40:18 Re: Patch for SonarQube code scan fixes.
Previous Message Aditya Toshniwal 2020-09-08 10:00:52 Re: Patch for SonarQube code scan fixes.