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

[Bug] ng add eslint-config-otter to support existing nested eslint configs #1695

Open
vscaiceanu-1a opened this issue Apr 22, 2024 · 0 comments
Open

[Bug] ng add eslint-config-otter to support existing nested eslint configs #1695

vscaiceanu-1a opened this issue Apr 22, 2024 · 0 comments
Labels
bug Something isn't working priority:low

Comments

@vscaiceanu-1a
Copy link
Member 

          What if it extends a config that extends '@o3r/eslint-config-otter'

Originally posted by @matthieu-crouzet in #1671 (comment)

Currently, if a eslintrc.json file extends a config that extends eslint-config-otter, the ng add script will add eslint-config-otter anyway.
Is this something to be fixed or should we not do any updates if an eslintrc.json file is found and encourage the user to do the setup manually?
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working priority:low
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@vscaiceanu-1a @pginoux-1A