Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

auto exclude gitignore patterns #99

Closed
twelvelabs opened this issue Nov 26, 2023 · 0 comments · Fixed by #109
Closed

auto exclude gitignore patterns #99

twelvelabs opened this issue Nov 26, 2023 · 0 comments · Fixed by #109
Labels
enhancement New feature or request

Comments

@twelvelabs
Copy link
Owner

If a gitignore file is present, we should auto-exclude any listed patterns.

  • There should be a config option to opt-out of this behavior.
    • We already need to add the context to PathIndexer, could pull the Config out of the context.
  • See: PathIndexer.exclude for where to hook place the logic.
  • See: https://github.com/sabhiram/go-gitignore
@twelvelabs twelvelabs added the enhancement New feature or request label Nov 26, 2023
@twelvelabs twelvelabs mentioned this issue Dec 26, 2023
Merged
@twelvelabs twelvelabs linked a pull request Dec 26, 2023 that will close this issue
feat: auto-exclude gitignore patterns #109
Merged
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat: auto-exclude gitignore patterns twelvelabs/stylist
1 participant
@twelvelabs