Although we try to use ignore pattern present by a .gitignore file, it's possible that users have a global git ignore settings or manually ignore a file.

We can use git command to list files when runs under a git worksapce,

This is a good idea, but it looks like the implementation in this PR only lists files that have been committed into the codebase, however, in most cases, I'm using the tool in the following workflow:

• In a git repo, checkout a new branch.
• Do modifications, for example, touch a-new-file.yaml.
• ... many other new files.
• license-eye h f to fix all files that are missing license headers.
• git add . && git commit -m'...'.

In this PR, nothing will be fixed, even if I run license-eye h f after git add ..

This also breaks users who used this tool as a git pre-commit hook.

@kezhenxu94 good point. Let me try to find a way to list staged/unstaged files.

@kezhenxu94 fixed at 18af4c4.

This patch fixes my case under incubator-kvrocks:

➜  incubator-kvrocks git: ~/Brittani/skywalking-eyes/bin/darwin/license-eye -c tools/ci/licenserc.yml header check
INFO Loading configuration from file: tools/ci/licenserc.yml 
INFO Totally checked 250 files, valid: 212, invalid: 0, ignored: 38, fixed: 0 
➜  incubator-kvrocks git: ~/go/bin/license-eye -c tools/ci/licenserc.yml header check
INFO Loading configuration from file: tools/ci/licenserc.yml 
INFO Totally checked 4279 files, valid: 212, invalid: 2045, ignored: 2022, fixed: 0 
INFO GITHUB_TOKEN is not set, license-eye won't comment on the pull request 
ERROR the following files don't have a valid license header: 
cmake-build-debug/_deps/gtest-src/googletest/scripts/test/Makefile
cmake-build-debug/_deps/luajit-src/src/Makefile
...
cmake-build-debug/_deps/lz4-src/contrib/snap/snapcraft.yaml 
ERROR one or more files does not have a valid license header

... where:

cat $(git config --global core.excludesFile)
# JetBrains
.idea
*.iml
cmake-build-*

# VS Code
.vscode

# macOS
.DS_Store

# Ruby
.ruby-version

I'll try to dig it out a bit :)

@kezhenxu94 I think we can handle the returned FileStatus. Previously, I assume that all ignored files don't occur in the status list, but it seems not true :)

@kezhenxu94 it's a go-git wart. Workaround at 2687d4e.

There're still some issues about .gitignore files in nested path go-git/go-git#522. But previously we also only handle the gitignore file at root path of the repo. If someone else asks for it, they can nudge the upstream fix or switch to a patching mod replacement. I'll leave it as is for now.

This patch fixes my case

I spot that it's about another issue on the master branch. I'll create an issue for it.

UPDATE - apache/skywalking#9560

it doesn't expand the tilde (~) for the core.excludesfile

I dug the code a bit. Finally, I notice that it's the Golang stdlib that causes the issue:

func main() {
	_, err := os.OpenFile("~/.gitignore", 0, 0666)
	fmt.Printf("%v", err)
}

// open ~/.gitignore: no such file or directory

Set up a cross-reference to go-git/go-git#578. If upstream resolves this issue, we can upgrade to fix the ~ in path issue.