suggestion: remove no-use-before-declare

[Martin Hochel]

I think this rule should be removed. It's not really necessary because no-var-keyword and also it's very slow to compute

Taken from TSlint docs:

Since most modern TypeScript doesn’t use var, this rule is generally discouraged and is kept around for legacy purposes. It is slow to compute, is not enabled in the built-in configuration presets, and should not be used to inform TSLint design decisions.

[Blake Embrey]

@Hotell This is relevant in more places than just variables, e.g. functions and classes. I'd prefer not to disable this, but also no-use-before-declare doesn't correctly implement https://eslint.org/docs/rules/no-use-before-define from ESLint. We need the ability to disable the options from out-of-scope. I'll disable until we can update the ESLint rules.

Reference from standard: https://github.com/standard/eslint-config-standard/blob/f7d50e01310866458fd844625cf588d4ee39823b/eslintrc.json#L151.

[Blake Embrey]

Created buzinas/tslint-eslint-rules#354 and released https://github.com/blakeembrey/tslint-config-standard/releases/tag/v8.0.1

[Martin Hochel]

looks good! thanks a lot @blakeembrey 🥇