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.

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

[eslint config] [imports] fix imports.js remove node_modules from imp… #2567

Draft
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

Hossein13M
Copy link

remove node_module from packages/eslint-config-airbnb-base/rules/imports.js

this pull request resolves the following issue: #2457

@ljharb
Copy link
Collaborator

ljharb commented Mar 7, 2022

This is a duplicate of #2464. Please do not close it; both PRs must now stay open until they can both be landed at once.

@ljharb ljharb marked this pull request as draft March 7, 2022 15:45
@Hossein13M
Copy link
Author

Any update on this? @ljharb
How about I merge #2464 into my branch and then add a co-authored commit so we can have both of these PRs?

@ljharb
Copy link
Collaborator

ljharb commented Sep 4, 2022

@Hossein13M since it's a semver-major change, and it's unlikely we'll have a semver-major change for a year or two, both PRs will need to stay open until that time.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants