I didn't replace all uses of var with const/let in the main module, because I believe there are some places where we're actually depending on var's function scope. That would be a nice future effort.
When this lands it'll require another major version bump since dropping commonjs is a breaking change.
I don't know if anyone is still looking at this repo but I'll leave this PR open for a little while in case there are any 👀
I didn't replace all uses of
var
withconst
/let
in the main module, because I believe there are some places where we're actually depending on var's function scope. That would be a nice future effort.When this lands it'll require another major version bump since dropping commonjs is a breaking change.
I don't know if anyone is still looking at this repo but I'll leave this PR open for a little while in case there are any 👀
Would love feedback.