-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Apply DeepScan recommended changes #77
Conversation
add me command
Update version to 1.3.14
Update version to 1.3.15
add topgg stats
Update version to 1.3.16
Update to version 1.3.17
add beta message
split the options at '
Prior to this commit, the unBan* commands returned `undefined`, but the function's return value was being treated as a boolean. Return statements have been added to make these work as intended.
changed the base branch from |
Node.js CI always fails on DeepSource is reporting issues w.r.t. certain variables not being defined. In practice, this isn't a problem because the way the codebase is organized, they'll always be defined by the time that code runs. I think they're useful, though, because it gives indicators of where the codebase could benefit from reorganization. DeepScan helped find some bugs in the unBan* commands due to the way certain promises were being resolved - pretty helpful! |
I wasn't able to test this remotely, due to lacking API keys etc. However, these were some relatively minor and simple-to-implement changes, so I don't expect any debugging is necessary.