Skip to content

recover-codegraphignore#531

Open
arkrolin wants to merge 1 commit into
colbymchenry:mainfrom
arkrolin:recover-codegraphignore
Open

recover-codegraphignore#531
arkrolin wants to merge 1 commit into
colbymchenry:mainfrom
arkrolin:recover-codegraphignore

Conversation

@arkrolin
Copy link
Copy Markdown

Thanks for the recent cleanup and streamlining of the codebase. I completely understand the motivation behind removing the bloated config.json and consolidating the exclusion logic into .gitignore. It makes the tool much simpler and more predictable for the vast majority of use cases.

However, relying exclusively on .gitignore creates a significant pain point for certain ecosystems, particularly for Go developers. As also raised by other community members in #511, there is a strong shared need for this. I'd like to propose bringing back a lightweight exclusion mechanism and am happy to submit a PR for it.

The Proposal (Addressing #511)

We need a way to decouple "files Git should track" from "files CodeGraph should index" to resolve the concerns here and in #511.

I would like to open a PR to reintroduce support for a .codegraphignore file.

  • I agree that the rest of the old config.json surface should remain deleted.
  • This PR would only restore the ability to add paths that the CodeGraph extractor should skip, supplementing the .gitignore logic rather than replacing it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant