-
Notifications
You must be signed in to change notification settings - Fork 14
Bug fixes amongst other improvements (huge PR) #5
base: master
Are you sure you want to change the base?
Conversation
Includes the following: - Naming all .format() variables - Add BotAdmin.clear command - Fix errors with variable names - Update "ignore" and "disable" command error messages - Add UTF-8 declaration to all files
This is very hard to go through and actually see what you changes since a large number of the changes are just you switching from spaces to tabs. I don't really know why you're doing that anyways. I'll try to sort through what I can soon. |
Aw shoot, my bad for the indent changes (I prefer tabs, as you can probably tell), apparently my Git client's "Ignore whitespace changes" doesn't actually work. >~< Changed it back to spaces so it's easier to go through! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is not needed, as apparently rauenzi's code preloads these cogs w/o notification
Ah, I see you have noticed and removed this line, no issues with me then
referring to Add basic cogs to Readme example
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why would these be added, you shouldn't run the bot inside the package directory, should you?
referring to Update .gitignore
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the logging module would be preferred here
referring to https://github.com/rauenzi/discordbot.py/pull/5/files#diff-c716f17f1fe5f8eb9dfbffa44ca9eb33R132
While working on my own Twyt Discord bot, there were a number of little fixes and updates I had performed. These updates include the following: