I noticed there are some modules that do not follow how upstream define variables. One example is the upload module, that uses the emacs-ssh-deploy package.
emacs-ssh-deploy assigns numbers 0 and 1 to some variables, like ssh-deploy-on-explicit-save.
The upload module instead uses t and nil for those same variables internally.
Is there a common practice in the Doom community to always prefer t and nil over upstream definitions?