Avoid hoisting

[Paul-Hebert]

After some internal discussion we decided to avoid relying on hoisting as it can be confusing.

Changes in /fixtures/ files can largely be ignored while reviewing.

[changeset-bot]

🦋 Changeset detected

Latest commit: 7c18624

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
@cloudfour/eslint-plugin	Major

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[calebeby]

I debated also changing the rule for classes and vars? But so far we haven't run into issues there so I left it alone.

I think we might as well change it for all of them. Not like we use them much anyways 😛

[calebeby]

Also, you changed the typescript version of the rule but not the built in version of the rule. The built in version of the rule is currently enabled by eslint-plugin-standard, which we extend. You can set the rule in the main config block so it is enabled for .js too (the built-in rule is disabled for .ts files in favor of the TS version of the rule).

[Paul-Hebert]

Thanks @calebeby , this is ready for another review!

[calebeby]

Thanks @Paul-Hebert!