Swag-Bot icon indicating copy to clipboard operation
Swag-Bot copied to clipboard

Published 20 hours ago • verified publisher icon c3duan

Modular Refactor

Open valkyrienyanko opened this issue 4 years ago • 4 comments

I can clean up index.js by spreading your functions / events across multiple files and folders. For e.g. ready.js message.js could be in a folder called Events. I also noticed your not using dotenv which could make handling secrets more convenient. Let me know if you want this and I'll get started when I have time.

valkyrienyanko avatar Apr 27 '20 07:04 valkyrienyanko

Sorry about the late reply. I do think my index.js is a bit clunky and if you are still interested in contributing, feel free to make a pull request. dotenv is also a good idea for handing environmental variables. Thanks for the help and suggestions.

c3duan avatar May 06 '20 22:05 c3duan

Alright I'll get started on it now then.

valkyrienyanko avatar May 06 '20 23:05 valkyrienyanko

The number of packages you have is a nightmare, I don't really want to install python so I'll PR over what I have. If you really want to look into modular functionality, take a look at this bot repo.

valkyrienyanko avatar May 07 '20 00:05 valkyrienyanko

See #14

valkyrienyanko avatar May 07 '20 00:05 valkyrienyanko