search

ddnet / ddnet-discordbot

10 stars 11 forks source link

pep and removing roles-channels in a separate file (config.py) +- made the code clearer #84

Closed ByFox213 closed 1 month ago

ByFox213 commented 6 months ago

At least there were so many f-string that are not being used.

Please review all the code that was changed and use the code I fixed:3

error search will help pylint or ruff https://pylint.readthedocs.io/ https://docs.astral.sh/ruff/tutorial/

in case of questions my discord: byfox

murpii commented 5 months ago

Oh dear, I've already made most of these changes. I really hope you're running your own instance of the bot for your projects; otherwise, it seems like you've invested time for nothing. :(

(Edit: Made most of these changes in the sense that I'm currently refactoring a good chunk of the bot at the moment.)

ByFox213 commented 5 months ago

Oh, I have my own api enabled bot (stormabot).

It took 4-6 hours to rewrite the partially original bot.

And the most interesting thing that happened was that at some point we started rewriting.

good luck rewriting)