Update should-i-commit-the-dependencies-in-my-vendor-directory.md

[reinink]

Added a third option for including vendor dependencies in main GIT repo.

Update: Just noticed a grammatical error on line 28. Wow, my very first pull request and I missed it up. 👎

[Seldaek]

You go to your patch-1 branch in your fork and edit the file again to fix the grammatical error. Aside from that, are you sure vendor/.git is working? I would think vendor/**/.git maybe or just .git would work.

[reinink]

@Seldaek Will do on the grammatical fix.

I've tested vendor/.git and it certainly works. I preferred that over .git, as it's clearer what the intention is, not to mention the confusion with the repo's main .git folder.

One note on this FAQs. I've tried the rm -rf vendor/**/.git command on OSX, and it does not work. However, rm -rf vendor/*/*/.git did work. I'm not overly familiar with the ** trick though, so I didn't want to make any changes here. Just thought I should mention it.

[Seldaek]

Yeah I think the double ** expansion doesn't work everywhere. But anyway it's really not the best idea.