feat: ignore gitignore & bazleignore files#56
feat: ignore gitignore & bazleignore files#56Siddhant-K-code wants to merge 1 commit intosourcegraph:mainfrom
Conversation
|
@Siddhant-K-code the CI job is failing. Could be on our end, can you confirm? |
|
I think, there is something wrong with the workflow. |
|
@slimsag any ideas? |
|
|
||
| // ignore .gitignore or .bazelignore file | ||
| if strings.Contains(indexName, ".gitignore") || strings.Contains(indexName, ".bazelignore") { | ||
| return nil, errors.New("files to be ignored") |
There was a problem hiding this comment.
I think for this to be useful we would need to look at / parse the .gitignore and .bazelignore files, then ignore those files, no?
Also, I think strings.Contains would be wrong here - we should ignore these at the time we produce the index, not inside of GetIndex.
Probbaly it would need to live inside the IndexDir methods of indexers themselves
doctree/doctree/indexer/indexer.go
Line 97 in 418506c
but I'm not 100% sure right now, may not have much time on my end to find the best place for this.
There was a problem hiding this comment.
Ah Yeah! This makes sense @slimsag
I will take a look at this 👀
permission to do so by an employer or client I am producing work for whom has this right.
fixes #33