Skip to content

ESLint Config Migration: Fixing lint errors for "no-use-before-define" rule.#1036

Merged
filmaj merged 1 commit intotslint-to-eslintfrom
no-use-before-define
Aug 5, 2021
Merged

ESLint Config Migration: Fixing lint errors for "no-use-before-define" rule.#1036
filmaj merged 1 commit intotslint-to-eslintfrom
no-use-before-define

Conversation

@filmaj
Copy link
Copy Markdown
Contributor

@filmaj filmaj commented Aug 5, 2021

Summary

This is a PR that should be merged into #1024 and incrementally addresses #842.

No ESLint config changes here - just getting the no-use-before-define rule passing.

Impact

Before

✖ 838 problems (649 errors, 189 warnings)
  111 errors and 0 warnings potentially fixable with the `--fix` option.

After

✖ 775 problems (586 errors, 189 warnings)
  109 errors and 0 warnings potentially fixable with the `--fix` option.

Requirements (place an x in each [ ])

@gitwave gitwave bot added the untriaged label Aug 5, 2021
@filmaj filmaj self-assigned this Aug 5, 2021
@filmaj filmaj added tests M-T: Testing work only and removed untriaged labels Aug 5, 2021
@filmaj filmaj merged commit aaff53b into tslint-to-eslint Aug 5, 2021
@filmaj filmaj deleted the no-use-before-define branch August 5, 2021 13:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tests M-T: Testing work only

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant