Formatting updates - Githubissues

Rixxan commented 2 years ago

A few minor improvements to Pydle's underlying code, for code efficiency and more structured application.

Included Changes:

Structuring and Ordering of Imports
Removal of Unused Imports
Updating of Whitespace to PEP8 Compliance
Removing unnecessary Pass statements (By request, also added ...)
Removal of raise-else and raise-return statements
Moved imports to top level
Improved bool comparisons (removed == and != to is and is not)
Removed arguments set to default value
Removed exception variable that was not used ~~- Marked additional async functions properly~~ Removed by request. Will include in own PR.
Reworded not x in y to x not in y for ease of reading
Condensed Dictionary Creation and Population
Removal of super() arguments

This update does not actually fundamentally change any returned values or functions.

Rixxan commented 2 years ago

I'd say yes to Black, reverted as requested. (But might be better to wait for 1.0)

theunkn0wn1 commented 2 years ago

As per conversation in side channel, let's hold off on black until the 1.0 release The delta for applying black will be huge.

shizmob / pydle