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

docs: Document homedir is a configuration root #15469

Merged
merged 1 commit into from Dec 31, 2021
Merged

Conversation

BBosman
Copy link
Contributor

@BBosman BBosman commented Dec 30, 2021

Fixes #14972

Some interest to make a PR was given by others in the past in the linked issue, but there hasn't been any activity from the original poster there after being asked for an update, so I decided to take a stab at fixing it. My apologies if that's considered inappropriate.

Prerequisites checklist

What is the purpose of this pull request? (put an "X" next to an item)

[X] Documentation update
[ ] Bug fix (template)
[ ] New rule (template)
[ ] Changes an existing rule (template)
[ ] Add autofix to a rule
[ ] Add a CLI option
[ ] Add something to the core
[ ] Other, please explain:

What changes did you make? (Give an overview)

Added some wording that when searching for configuration files the ~/ directory is also considered a root and searching will stop there. It also mentions that configuration files present in that directory will be ignored with the removal of support for Personal Configuration files.

Is there anything you'd like reviewers to focus on?

Nothing specific.

@eslint-github-bot eslint-github-bot bot added triage documentation labels Dec 30, 2021
nzakas
nzakas approved these changes Dec 31, 2021
Copy link
Member

@nzakas nzakas left a comment

LGTM. Thanks.

@nzakas
Copy link
Member

@nzakas nzakas commented Dec 31, 2021

Closes #14972

@nzakas nzakas merged commit 6c4dee2 into eslint:main Dec 31, 2021
14 checks passed
@BBosman BBosman deleted the userroot branch Dec 31, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation triage
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants