Ash515 / AshTech-VoiceAssistant

AI Gadget Management System💻📳📱
MIT License
45 stars 39 forks source link

Pep #102

Closed vybhav72954 closed 3 years ago

vybhav72954 commented 3 years ago

Related Issuse or bug

Fixes: #100

Describe the changes you've made

As you can see in the code, I have formatted the code and enforced proper PEP-8 guidelines all across the codebase.

Type of change

Please delete options that are not relevant.

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce.

I am planning to run another static analysis, I will upload the result once it is done, you can wait for it before merging this to see the effect.

Describe if there is any unusual behaviour of your code(Write NA if there isn't)

NA

Checklist:

Screenshots

Original Updated
original screenshot updated screenshot

I will upload proof of concept ss within some time in the comments.

vybhav72954 commented 3 years ago

Errors reported in static analysis before the PEP-8 Guidelines enforced CLICK HERE

Errors reported in static analysis after the PEP-8 Guidelines enforced CLICK HERE

PyLint Score before - 8.1 PyLint Score after - 8.9 (Pep Guidelines are effective)

I by mistake made a PR #103, but I have used it as a proof of concept now, so don't merge it at all, but it shows a nice picture.

The Linters that I have made are the ones I suggested in #101