search

rHomelab / LabBot-Cogs

Cogs for the Red-based Homelab Discord server bot.
GNU General Public License v3.0
36 stars 14 forks source link

Remove line break #248

Closed S33G closed 1 year ago

tigattack commented 1 year ago

See https://peps.python.org/pep-0008/#blank-lines

rippleFCL commented 1 year ago

this is a little miss spoken. the linting specs in python is currently defined in PEP (python enhancement proposal) 8 please see this link HERE. within section CODE LAYOUT subsection BLANK LINE. thank you @tigattack for commenting faster but i do fully belive this answer gose into more indepth. and should be prioritised when reading. the section mentioned previously quite SPECIFICLY describes Surround top-level function and class definitions with two blank lines. and as that function is a top-level async function. it should specificly be surrounded by a double line break making this pull request redundant and invalid