Install Jinja2 templates for click_completion

[wjv]

The new version of click_completion requires its Jinja2 templates to be installed along with the package in order to work. If not, any invocation of pipenv --completion results in a jinja2.exceptions.TemplateNotFound for the template in question.

I’ve just trivially added the pattern to setup.py, and tested that it works.

[uranusjr]

@techalchemy We’ll need another release very soon 🤦‍♂️

[techalchemy]

Today is the plan

[ilovezfs]

Is this sufficient to make sure they end up in the sdist too?

[uranusjr]

@ilovezfs It should, I think.

[daogilvie]

@ilovezfs @uranusjr Hi — sadly it looks like the latest commit of master, with this PR merged, does not include the files in the sdist under setuptools 39.0.1, as I am just discovering now 😂 . I don't really grok Python packaging sometimes, but I have found adding include pipenv/vendor/click_completion/*.j2 to the MANIFEST.in solves the problem. I can reopen an issue and/or make a PR, but I don't want to just assume you are happy with the solution of adding to the MANIFEST.in file.

For completeness I have tried CPython 3.6.5 and 2.7.15 on MacOS high sierra. The sdist as generated there is affected, as is the one installed by homebrew on my system.

Please advise if I'm somehow using a bad branch, but the merge commit for this PR is in the git log so I'm pretty sure it's in there.

[techalchemy]

We didn’t actually release an sdist yet but I would believe that our manifest doesn’t include these. Feel free to PR a fix and thanks for the heads up!