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: double negative in configuration file resolution #18418

Closed
1 task done
MikeMcC399 opened this issue May 5, 2024 · 2 comments · Fixed by #18419
Closed
1 task done

Docs: double negative in configuration file resolution #18418

MikeMcC399 opened this issue May 5, 2024 · 2 comments · Fixed by #18419
Labels
accepted There is consensus among the team that this change meets the criteria for inclusion documentation Relates to ESLint's documentation

Comments

@MikeMcC399
Copy link
Contributor

Docs page(s)

What documentation issue do you want to solve?

Both pages contain the sentence:

If none of the files are not found, it checks the parent directory for each file.

This is a double negative and it does not make sense the way it is written.

What do you think is the correct solution?

Remove the word "not", so the sentence becomes:

If none of the files are found, it checks the parent directory for each file.

Participation

  • I am willing to submit a pull request for this change.

Additional comments

No response

@MikeMcC399 MikeMcC399 added the documentation Relates to ESLint's documentation label May 5, 2024
@fasttime
Copy link
Member

fasttime commented May 5, 2024

Thanks for spotting this @MikeMcC399! I don't think we can fix the wording in the v8 docs at this time, because that would require doing a new v8.x release, which isn't planned. But feel free to submit a pull request to fix the v9 docs. The file to edit is docs/src/use/configure/configuration-files.md.

@fasttime fasttime added the accepted There is consensus among the team that this change meets the criteria for inclusion label May 5, 2024
@MikeMcC399
Copy link
Contributor Author

@fasttime

Thanks for confirming!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accepted There is consensus among the team that this change meets the criteria for inclusion documentation Relates to ESLint's documentation
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

2 participants