Add other ignorefile sample #1253
Conversation
There was a problem hiding this comment.
Code Review
This pull request adds a new sample ignore file for testing purposes. The patterns included are generally well-suited for testing various ignore rules. I have provided a couple of suggestions to improve some of the patterns to better align with common best practices for .gitignore files, which will make them more robust and explicit. These changes involve adding trailing slashes to directory patterns and adjusting a pattern to ensure an entire directory's contents are ignored recursively.
| /dist | ||
| /build |
There was a problem hiding this comment.
For clarity and to ensure only directories are matched, it's a good practice to add a trailing slash to directory patterns. This prevents accidentally ignoring a file that might have the same name as the directory. For example, mydir/ matches the directory mydir but not a file named mydir, whereas mydir would match both.
/dist/
/build/
related to: #662 and #1249