Commit 9cab93c0 by Igor Djordjevic

ignore: improve `git_ignore_path_is_ignored` description Git analogy

In attempt to provide adequate Git command analogy in regards to
ignored files handling, `git_ignore_path_is_ignored` description
mentions doing `git add .` on directory containing the file, and
whether the file in question would be added or not - but behavior of
the two matches for untracked files only, making the comparison
misleading in general sense.

For tracked files, Git doesn't subject them to ignore rules, so even
if a rule applies, `git add .` would actually add the tracked file
changes to index, while `git_ignore_path_is_ignored` would still
consider the file being ignored (as it doesn't check the index, as
documented).

Let's provide `git check-ignore --no-index` as analogous Git command
example instead, being more aligned with what `git_ignore_path_is_ignored`
is about, no matter if the file in question is already tracked or not.

See issue #4720 (git_ignore_path_is_ignored documentation
misleading?, 2018-07-10)[1] for additional information.

[1] https://github.com/libgit2/libgit2/issues/4720
parent 6dfc8bc2
...@@ -59,8 +59,8 @@ GIT_EXTERN(int) git_ignore_clear_internal_rules( ...@@ -59,8 +59,8 @@ GIT_EXTERN(int) git_ignore_clear_internal_rules(
* given file. This indicates if the file would be ignored regardless of * given file. This indicates if the file would be ignored regardless of
* whether the file is already in the index or committed to the repository. * whether the file is already in the index or committed to the repository.
* *
* One way to think of this is if you were to do "git add ." on the * One way to think of this is if you were to do "git check-ignore --no-index"
* directory containing the file, would it be added or not? * on the given file, would it be shown or not?
* *
* @param ignored boolean returning 0 if the file is not ignored, 1 if it is * @param ignored boolean returning 0 if the file is not ignored, 1 if it is
* @param repo a repository object * @param repo a repository object
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment