Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Update .gitignore to add /doc/tags #25

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

arcseldon
Copy link

/doc/tags gets auto-created upon plugin install (i am using Vundle plugin manager, believe this is vim behaviour). When using git submodules, the existence of doc/tags causes git errors indicating untracked content. Noticed that many other vim plugins account for this, and add /doc/tags to their .gitignore files -
for example: vim-fugitive, or tabular. Have about 60 plugins, and only 4 currently missing /doc/tags in their .gitignore - please consider accepting this PR.

`/doc/tags` gets auto-created upon plugin install (i am using Vundle plugin manager, believe this is vim behaviour). When using git submodules, the existence of `doc/tags` causes git errors indicating untracked content.
Noticed that many other vim plugins account for this, and add `tags` or `doc/tags` to their .gitignore files -
for example: [vim-fugitive](https://github.com/tpope/vim-fugitive/blob/master/.gitignore), [nerdcommenter](https://github.com/scrooloose/nerdcommenter/blob/master/.gitignore) or [tabular](https://github.com/godlygeek/tabular/blob/master/.gitignore).  Have about 60 plugins, and only 4 currently missing `tags` in their .gitignore - please consider accepting this PR.
@arcseldon
Copy link
Author

Hi. Any update on getting this PR approved?

@arcseldon
Copy link
Author

@szw - Hello. Any chance you could please get this PR completed?

If you have any concerns / reasons to reject it, please can you update here. All the other plugins that were missing /doc/tags have now accepted the PR. Thanks for any help accommodating this request.

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