Re: Patch for SonarQube code scan fixes.
От | Aditya Toshniwal |
---|---|
Тема | Re: Patch for SonarQube code scan fixes. |
Дата | |
Msg-id | CAM9w-_mFY1t0cC8pK3aBSzScW9C16xvLFsU1HczYaoezAypfOQ@mail.gmail.com обсуждение исходный текст |
Ответ на | Re: Patch for SonarQube code scan fixes. (Dave Page <dpage@pgadmin.org>) |
Ответы |
Re: Patch for SonarQube code scan fixes.
|
Список | pgadmin-hackers |
Hi Dave,
On Tue, Sep 8, 2020 at 3:53 PM Dave Page <dpage@pgadmin.org> wrote:
On Tue, Sep 8, 2020 at 11:01 AM Aditya Toshniwal <aditya.toshniwal@enterprisedb.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 :-)
I remember Akshay had fixed all the PEP8 issues in pgcli and sent the patch. But it was rejected by the pgcli author. :/
On Tue, Sep 8, 2020 at 2:59 PM Nikhil Mohite <nikhil.mohite@enterprisedb.com> wrote:Hi Team,I have fixed some code smell issues in the SonarQube scan, PFA patch.Details as follows:1. psycopg2/connection:2. psycopg2/server_manager:
- 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.
3. sqlautocomplete/parseutils:
- 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.
- Refactor this function to reduce its Cognitive Complexity from 23 to the 15 allowed.
----Thanks,Aditya ToshniwalpgAdmin hacker | Sr. Software Engineer | edbpostgres.com"Don't Complain about Heat, Plant a TREE"--
Thanks,
Aditya Toshniwal
pgAdmin hacker | Sr. Software Engineer | edbpostgres.com
"Don't Complain about Heat, Plant a TREE"
В списке pgadmin-hackers по дате отправления: