Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[import/extensions] ".." and "." import paths are incorrectly flagged as requiring an extension, when either is al local package #2844

Open
benasher44 opened this issue Jul 31, 2023 · 2 comments
Open

[import/extensions] ".." and "." import paths are incorrectly flagged as requiring an extension, when either is al local package #2844

benasher44 opened this issue Jul 31, 2023 · 2 comments
Labels
bug help wanted

Comments

@benasher44
Copy link
Contributor

In #2778, we determined that this plugin incorrectly flags ".." and "." import paths, when those imports refer to a local package (with a package.json).
@benasher44 benasher44 mentioned this issue Jul 31, 2023
Merged
@ljharb
Copy link
Member

ljharb commented Aug 5, 2023

@benasher44 a failing test case would be most helpful :-)

@benasher44
Copy link
Contributor Author

Isn't that these? #2778 (comment)

Basically they should be failing, so ideally ones makes a PR that makes these fail and then updates the tests

kevinoid added a commit to kevinoid/node-project-template that referenced this issue Sep 3, 2023
@kevinoid
Disable import/extensions for require('..')
de86f5e 
To work around
import-js/eslint-plugin-import#2844

Signed-off-by: Kevin Locke <kevin@kevinlocke.name>
kevinoid added a commit to kevinoid/git-branch-is that referenced this issue Sep 5, 2023
@kevinoid
Disable import/extensions for require('..')
501c5d3 
To work around
import-js/eslint-plugin-import#2844

Signed-off-by: Kevin Locke <kevin@kevinlocke.name>
kevinoid added a commit to kevinoid/json-replace-exponentials that referenced this issue Sep 5, 2023
@kevinoid
Disable import/extensions for require('..')
bb9fadf 
To work around
import-js/eslint-plugin-import#2844

Signed-off-by: Kevin Locke <kevin@kevinlocke.name>
kevinoid added a commit to kevinoid/json-stringify-raw that referenced this issue Sep 5, 2023
@kevinoid
Disable import/extensions for require('..')
bdb4c55 
To work around
import-js/eslint-plugin-import#2844

Signed-off-by: Kevin Locke <kevin@kevinlocke.name>
kevinoid added a commit to kevinoid/nodecat that referenced this issue Sep 5, 2023
@kevinoid
Disable import/extensions for require('..')
9bb6765 
To work around
import-js/eslint-plugin-import#2844

Signed-off-by: Kevin Locke <kevin@kevinlocke.name>
kevinoid added a commit to kevinoid/openapi-transformer-base that referenced this issue Sep 5, 2023
@kevinoid
Disable import/extensions for require('..')
dc8fdbb 
To work around
import-js/eslint-plugin-import#2844

Signed-off-by: Kevin Locke <kevin@kevinlocke.name>
kevinoid added a commit to kevinoid/promise-nodeify that referenced this issue Sep 5, 2023
@kevinoid
Disable import/extensions for require('..')
dbb66bb 
To work around
import-js/eslint-plugin-import#2844

Signed-off-by: Kevin Locke <kevin@kevinlocke.name>
kevinoid added a commit to kevinoid/swagger-spec-validator that referenced this issue Sep 5, 2023
@kevinoid
Disable import/extensions for require('..')
6973247 
To work around
import-js/eslint-plugin-import#2844

Signed-off-by: Kevin Locke <kevin@kevinlocke.name>
kevinoid added a commit to kevinoid/travis-status that referenced this issue Sep 5, 2023
@kevinoid
Disable import/extensions for require('..')
71dece0 
To work around
import-js/eslint-plugin-import#2844

Signed-off-by: Kevin Locke <kevin@kevinlocke.name>
kevinoid added a commit to kevinoid/stream-compare that referenced this issue Sep 6, 2023
@kevinoid
Disable import/extensions for require('..')
efef1ad 
To work around
import-js/eslint-plugin-import#2844

Signed-off-by: Kevin Locke <kevin@kevinlocke.name>
kevinoid added a commit to kevinoid/promised-read that referenced this issue Sep 6, 2023
@kevinoid
Disable import/extensions for require('..')
66fca5b 
To work around
import-js/eslint-plugin-import#2844

Signed-off-by: Kevin Locke <kevin@kevinlocke.name>
kevinoid added a commit to kevinoid/inflate-auto that referenced this issue Sep 15, 2023
@kevinoid
Disable import/extensions for require('..')
3a606a9 
To work around
import-js/eslint-plugin-import#2844

Signed-off-by: Kevin Locke <kevin@kevinlocke.name>
kevinoid added a commit to kevinoid/json-replace-exponentials that referenced this issue Oct 1, 2023
@kevinoid
Disable import/extensions for require('..')
f10e997 
To work around
import-js/eslint-plugin-import#2844

Signed-off-by: Kevin Locke <kevin@kevinlocke.name>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug help wanted
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ljharb @benasher44