Skip to content

ESLint Config Migration: Fix import/order lint rules failing#1031

Merged
filmaj merged 1 commit intotslint-to-eslintfrom
import-order
Aug 4, 2021
Merged

ESLint Config Migration: Fix import/order lint rules failing#1031
filmaj merged 1 commit intotslint-to-eslintfrom
import-order

Conversation

@filmaj
Copy link
Copy Markdown
Contributor

@filmaj filmaj commented Aug 4, 2021

Summary

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

No ESLint config changes here - just getting the import/order rule passing. Nothing too controversial in here!

Impact

#1024 lint failures progress as of posting this PR:

✖ 927 problems (737 errors, 190 warnings)
  187 errors and 0 warnings potentially fixable with the `--fix` option.

Failures with this PR included:

✖ 903 problems (713 errors, 190 warnings)
  163 errors and 0 warnings potentially fixable with the `--fix` option.

Requirements (place an x in each [ ])

@gitwave gitwave bot added the untriaged label Aug 4, 2021
@filmaj filmaj self-assigned this Aug 4, 2021
@filmaj filmaj added enhancement M-T: A feature request for new functionality tests M-T: Testing work only and removed untriaged labels Aug 4, 2021
@filmaj filmaj merged commit bdc5efb into tslint-to-eslint Aug 4, 2021
@filmaj filmaj changed the title [WIP] ESLint Config Migration: Fix import/order lint rules failing ESLint Config Migration: Fix import/order lint rules failing Aug 5, 2021
@WilliamBergamin WilliamBergamin deleted the import-order branch May 20, 2025 19:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement M-T: A feature request for new functionality tests M-T: Testing work only

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant