Repository navigation
4. Additional Developer Notes
Please see the discord.js API and guides located at https://discord.js.org/#/
Including addons: Bookmarks, Bracket Pair Colorizer, ESLint, Github build status, Gitlens - Git supercharged.
app.js - base startup and bot logic database.js - logic for working with persistent/stored data in the mysql database generator.js - logic for providing peronalised error messages to end users commands/*.js - logic for all different types of commands. Each file represents commands specific to a role/permission level.
Please document all functions as specified by http://usejsdoc.org/ Furthermore, feel free to include inline comments to clarify any program logic or longer functions
Several global variables are used to keep track of certain configuration values or constants throughout the program. As a general rule, global variables should be avoided, but if you must make use of a global, please redefine it as a local at the top of the file like
logger = global.logger
so it is easy to see what globals your module/file depends on.
Overall, all commits/additions to the code are subject to code review from a developer with merge permissions from the main repo. (Usually the CTO or a council member). Before asking for code review you should always keep in mind... Is my code clear? Is my code well commented? Is anything in my code hacky? Is there any errors I haven't accounted for? Have I tested this code on my own? If any of these questions is not a resounding yes, you should reconsider asking for code review.