While upgrading dependencies and testing the bot, I noticed a couple issues that needed addressing
anything that checked permissions was broken by the upgrade to v13, because Member.hasPermission was removed
timer registration was broken since the candidates type was now a Collection and not an Array
Errors during shutdown wouldn't be handled right, because client.destroy() hasn't returned a promise in quite a long time
and setting process.exitCode doesn't trigger an immediate exit
the logic for saving was really gnarly and needed straightening out
I also update the allowed language syntax to near-current, since the nodeJS requirement is very recent. I didn't use these new language features (such as optional chaining) in this PR though, to limit churn.
While upgrading dependencies and testing the bot, I noticed a couple issues that needed addressing
anything that checked permissions was broken by the upgrade to v13, because
Member.hasPermission
was removedtimer registration was broken since the
candidates
type was now a Collection and not an ArrayErrors during shutdown wouldn't be handled right, because
client.destroy()
hasn't returned a promise in quite a long timethe logic for saving was really gnarly and needed straightening out
I also update the allowed language syntax to near-current, since the nodeJS requirement is very recent. I didn't use these new language features (such as optional chaining) in this PR though, to limit churn.