Skip to content
This repository has been archived by the owner on Nov 10, 2022. It is now read-only.

Clarify default ignoring of node_modules #456

Closed
wants to merge 1 commit into from
Closed

Clarify default ignoring of node_modules #456

wants to merge 1 commit into from

Conversation

matijs
Copy link

@matijs matijs commented Mar 15, 2018

When using ESLint from the root of a mono repo node_modules needs to be explicitly added to .eslintignore for ESLint to ignore for example packages/*/node_modules. Clarified this in the configuring documentation.

When using ESLint from the root of a mono repo `node_modules` needs to be explicitly added to `.eslintignore` for ESLint to ignore for example `packages/*/node_modules`. Clarified this in the configuring documentation.
@jsf-clabot
Copy link

jsf-clabot commented Mar 15, 2018

CLA assistant check
All committers have signed the CLA.

@eslint-deprecated eslint-deprecated bot added the triage An ESLint team member will look at this issue soon label Mar 15, 2018
@not-an-aardvark
Copy link
Member

Thanks for the PR. This page is actually autogenerated from the corresponding page in the eslint/eslint repository; can you open a PR to that repo instead? (If we merge this PR, it will get overwritten the next time we do a release.)

@matijs
Copy link
Author

matijs commented Mar 15, 2018

@not-an-aardvark oh that’s silly of me will do!

@matijs matijs closed this Mar 15, 2018
@matijs
Copy link
Author

matijs commented Mar 16, 2018

Opened a new PR in the eslint/eslint repo.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
triage An ESLint team member will look at this issue soon
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants