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

Adding MANIFEST.in to repository. #9

Merged
merged 1 commit into from
Jun 22, 2017
Merged

Conversation

dhermes
Copy link
Contributor

@dhermes dhermes commented Jun 22, 2017

Fixes #8.

/cc @johanneskoester

@jonparrott I mostly just copy-pasted the MANIFEST.in from Datastore. As a result I am somewhat unsure, so can you "scrutinize"?

MANIFEST.in Outdated
@@ -0,0 +1,4 @@
include README.rst LICENSE
recursive-include google
recursive-include unit_tests *

This comment was marked as spam.

@dhermes
Copy link
Contributor Author

dhermes commented Jun 22, 2017

OK I rebased this after #10 and #11 went in, hopefully we can get green CI and merge.

@dhermes
Copy link
Contributor Author

dhermes commented Jun 22, 2017

OK the latest failure is due to the fact that PRs don't have access to env. vars. I'm just merging (sorry for laziness, I just have too much else going on. If there is a breakage in the "merge" build, I'll look into it)

@dhermes dhermes merged commit 3625888 into googleapis:master Jun 22, 2017
@dhermes dhermes deleted the fix-8 branch June 22, 2017 19:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants