add variable to ignore specific projects #1916
Draft
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I want to add a git repo to my home directory in order to track dotfiles (I'm aware there are other approaches that don't involve creating
~/.git/
, but for reasons beyond the scope of this PR, I want to do it this way). When I do this, in doom-emacsdoom doctor
complainsMaybe I'm misunderstanding something, but it seems like the resolution here isn't something I want to do. I still want projectile to recognize dirs with
.git
as project roots, I just want it to not consider$HOME
to be the universal project root, as threatened bydoom doctor
. Therefore, I need it to say that one or more named directories are not project roots, regardless of what the root-determination machinery would say otherwise.So I add a variable to hold a list of such directories, and consult it inside
projectile-locate-dominating-file
.NOTE: the PR is intentionally missing some required cleanup (like comments) in case there is something fundamentally wrong with my whole approach (I suck at elisp). That's why it's marked as a draft. AFAIK, the code itself does what it should, but I could be wrong.
Before submitting a PR make sure the following things have been done (and denote this
by checking the relevant checkboxes):
eldev test
)M-x checkdoc
warningsThanks!