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

[Fix] no-extraneous-dependencies: fix package name algorithm - #2097

Merged
ljharb merged 1 commit into
import-js:masterfrom
paztis:master
May 27, 2021
Merged

[Fix] no-extraneous-dependencies: fix package name algorithm#2097
ljharb merged 1 commit into
import-js:masterfrom
paztis:master

Conversation

@paztis

@paztis paztis commented May 25, 2021 •
edited by ljharb
Loading

Copy link
Copy Markdown
  • resolve nested package.json problems (a/b/c import will check a, a/b and a/b/c)
  • resolve renamed dependencies: checks the import name and the resolve package name

Fixes #2066. Fixes #2065. Fixes #2058. Fixes #2078.

@paztis @ljharb
[Fix] no-extraneous-dependencies: fix package name algorithm
da8d584 
- resolve nested package.json problems (a/b/c import will check a, a/b and a/b/c)
- resolve renamed dependencies: checks the import name and the resolve package name

Fixes import-js#2066. Fixes import-js#2065. Fixes import-js#2058. Fixes import-js#2078.
This was referenced May 25, 2021
Closed
Closed
Closed
@coveralls

coveralls commented May 25, 2021

Copy link
Copy Markdown

Coverage Status

Coverage increased (+1.04%) to 82.53% when pulling d7119ef on paztis:master into 20c373c on benmosher:master.

2 similar comments
@coveralls

coveralls commented May 25, 2021

Copy link
Copy Markdown

Coverage Status

Coverage increased (+1.04%) to 82.53% when pulling d7119ef on paztis:master into 20c373c on benmosher:master.

@coveralls

coveralls commented May 25, 2021

Copy link
Copy Markdown

Coverage Status

Coverage increased (+1.04%) to 82.53% when pulling d7119ef on paztis:master into 20c373c on benmosher:master.

@coveralls

coveralls commented May 25, 2021 •
edited
Loading

Copy link
Copy Markdown

Coverage Status

Coverage increased (+1.04%) to 82.584% when pulling da8d584 on paztis:master into 81b9d24 on benmosher:master.

ljharb
ljharb requested changes May 25, 2021
View reviewed changes

@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.

The fix looks good, but we'll need to add some regression tests, so this can't happen again.

Comment thread src/rules/no-extraneous-dependencies.js
@ljharb ljharb changed the title [Defect 2066] fix extraneous-dependencies algorithm [Fix] extraneous-dependencies: fix package name algorithm May 25, 2021
@paztis

paztis commented May 26, 2021

Copy link
Copy Markdown
Author

unit tests added

@mtnowl mtnowl mentioned this pull request May 27, 2021
Closed
@ljharb ljharb changed the title [Fix] extraneous-dependencies: fix package name algorithm [Fix] no-extraneous-dependencies: fix package name algorithm May 27, 2021
@ljharb
ljharb merged commit da8d584 into import-js:master May 27, 2021
@tarlepp tarlepp mentioned this pull request May 29, 2021
Closed
@erezrokah erezrokah mentioned this pull request May 30, 2021
Merged
1 task
@tw0517tw tw0517tw mentioned this pull request Jun 1, 2021
Merged
@ekasbohm ekasbohm mentioned this pull request Jun 1, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment