Sitelet https://github.com/import-js/eslint-plugin-import/pull/2226
Skip to content

[Fix] order: Fix import ordering in TypeScript module declarations - #2226

Merged
ljharb merged 1 commit into
import-js:mainfrom
remcohaszing:fix-order-typescript-module-declarations
Sep 18, 2021
Merged

ljharb merged 1 commit into
import-js:mainfrom
remcohaszing:fix-order-typescript-module-declarations

Conversation

@remcohaszing

Copy link
Copy Markdown
Contributor

Without this, import/order checks if all imports in a file are sorted. The autofix would then move all imports to the type of the file, breaking TypeScript module declarations.

Closes #2217

Without this, `import/order` checks if all imports in a file are sorted. The
autofix would then move all imports to the type of the file, breaking TypeScript
module declarations.

Closes #2217
@codecov

codecov Bot commented Sep 12, 2021 •

Copy link
Copy Markdown

Codecov Report

Merging #2226 (7f25ace) into main (4ed7867) will increase coverage by 0.24%.
The diff coverage is 100.00%.

❗ Current head 7f25ace differs from pull request most recent head 47ea669. Consider uploading reports for the commit 47ea669 to get more accurate results
Impacted file tree graph

@@            Coverage Diff             @@
##             main    #2226      +/-   ##
==========================================
+ Coverage   84.04%   84.29%   +0.24%     
==========================================
  Files          96       93       -3     
  Lines        3034     2979      -55     
  Branches      900      882      -18     
==========================================
- Hits         2550     2511      -39     
+ Misses        484      468      -16     
Impacted Files Coverage Δ
src/rules/order.js 99.06% <100.00%> (-0.30%) ⬇️
src/ExportMap.js 92.85% <0.00%> (ø)
config/typescript.js 100.00% <0.00%> (ø)
utils/unambiguous.js 100.00% <0.00%> (ø)
src/rules/no-unresolved.js 100.00% <0.00%> (ø)
utils/visit.js
utils/pkgUp.js
utils/readPkgUp.js
src/rules/no-unused-modules.js 99.76% <0.00%> (+2.32%) ⬆️
... and 1 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4ed7867...47ea669. Read the comment docs.

@ljharb ljharb left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seems great!

Comment thread src/rules/order.js Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

import/order false positives for TypeScript module declarations

2 participants