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: gitignore syntax updates (fixes #8139) #10776
Docs: gitignore syntax updates (fixes #8139) #10776
Conversation
# Please enter the commit message for your changes. Lines starting # with '#' will be ignored, and an empty message aborts the commit. # # On branch gitignore-syntax-updates # Changes to be committed: #modified: docs/user-guide/command-line-interface.md #modified: docs/user-guide/configuring.md # Adds cross links between .eslintignore and --ignore-pattern sections. Adds link to --ignore-pattern section directing to .gitignore specs. Adds description to .eslintignore section that emphasizes forward slashes being necessary with examples of valid and invalid syntax. Adds explicit statements to both sections that support this example.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks! Left one minor comment but it's not a blocker.
docs/user-guide/configuring.md
Outdated
* Lines preceded by `!` are negated patterns that re-include a pattern that was ignored by an earlier pattern. | ||
* Ignore patterns behave according to the `.gitignore` [specification](https://git-scm.com/docs/gitignore). | ||
|
||
Of particular note is that like `.gitignore` files all paths used as patterns for both `.eslintignore` and `--ignore-pattern` must use forward slashes as their path separators. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nitpick: Could you add a comma after "like .gitignore files"?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I welcome the nitpick and appreciate it. I am surprised I missed it even! Going to add now.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks!
I'm going to leave this open for a couple of days in case others wish to review.
Thanks for contributing! |
What is the purpose of this pull request? (put an "X" next to item)
[X ] Documentation update
[ ] Bug fix (template)
[ ] New rule (template)
[ ] Changes an existing rule (template)
[ ] Add autofixing to a rule
[ ] Add a CLI option
[ ] Add something to the core
[ ] Other, please explain:
What changes did you make? (Give an overview)
Adds cross links between .eslintignore and --ignore-pattern sections.
Adds link to --ignore-pattern section directing to .gitignore specs.
Adds description to .eslintignore section that emphasizes forward slashes being necessary with examples of valid and invalid syntax.
Adds explicit statements to both sections that support this example.
Is there anything you'd like reviewers to focus on?
Ensuring that changes are clear and that they keep original entries' context.