Re: [pgAdmin][SonarQube] Cognitive complexity - Mailing list pgadmin-hackers

From Aditya Toshniwal
Subject Re: [pgAdmin][SonarQube] Cognitive complexity
Date
Msg-id CAM9w-_k752p24RTGCfJNt6ugzpb_+DP1aTEGHRssahPVAzo2tg@mail.gmail.com
Whole thread Raw
In response to Re: [pgAdmin][SonarQube] Cognitive complexity  (Akshay Joshi <akshay.joshi@enterprisedb.com>)
Responses Re: [pgAdmin][SonarQube] Cognitive complexity  (Akshay Joshi <akshay.joshi@enterprisedb.com>)
List pgadmin-hackers
Hi Akshay,

Attached is the updated patch.

On Tue, Aug 25, 2020 at 11:38 AM Akshay Joshi <akshay.joshi@enterprisedb.com> wrote:
Hi Aditya

Please fix the following issues:
  • Add docstrings to the newly created functions.
  • Fix PEP8 issue.


On Mon, Aug 24, 2020 at 5:02 PM Aditya Toshniwal <aditya.toshniwal@enterprisedb.com> wrote:
Hi Hackers,

Attached patch reduces the cognitive complexity of functions for below:
web/.../schemas/packages/edbfuncs/__init__.py 21 to 15
web/.../databases/schemas/types/__init__.py 39 to 15; 80 to 15
web/.../server_groups/servers/pgagent/__init__.py 39 to 15
web/pgadmin/misc/bgprocess/process_executor.py 19 to 15
web/pgadmin/tools/backup/__init__.py 54 to 15

Please review.

--
Thanks,
Aditya Toshniwal
pgAdmin hacker | Sr. Software Engineer | edbpostgres.com
"Don't Complain about Heat, Plant a TREE"


--
Thanks & Regards
Akshay Joshi
pgAdmin Hacker | Sr. Software Architect
EDB Postgres
Mobile: +91 976-788-8246



--
Thanks,
Aditya Toshniwal
pgAdmin hacker | Sr. Software Engineer | edbpostgres.com
"Don't Complain about Heat, Plant a TREE"
Attachment

pgadmin-hackers by date:

Previous
From: Akshay Joshi
Date:
Subject: Re: [pgAdmin][SonarQube] Cognitive complexity
Next
From: Akshay Joshi
Date:
Subject: pgAdmin 4 commit: Fixed cognitive complexity issues reported by SonarQu