Skip to content

chore(dist): add test data - #916

Merged
max-wittig merged 1 commit into
masterfrom
chore/add-test-to-dist
Oct 22, 2019
Merged

chore(dist): add test data#916
max-wittig merged 1 commit into
masterfrom
chore/add-test-to-dist

Conversation

@bufferoverflow

@bufferoverflow bufferoverflow commented Oct 22, 2019

Copy link
Copy Markdown
Member

Closes #907

@max-wittig

max-wittig commented Oct 22, 2019

Copy link
Copy Markdown
Member

Did you test this? I think we are already in the gitlab dir.

@bufferoverflow

bufferoverflow commented Oct 22, 2019

Copy link
Copy Markdown
Member Author 
$ python3 setup.py sdist bdist_wheel
$ tar -tvzf dist/python-gitlab-1.12.1.tar.gz | grep json
-rw-r--r--  0 meierro staff    2615 Oct 17 11:05 python-gitlab-1.12.1/gitlab/tests/data/todo.json

@bufferoverflow

bufferoverflow commented Oct 22, 2019

Copy link
Copy Markdown
Member Author

@max-wittig I just figured out that the file is not within whl and egg file, I'm unsure if it should be there or not.

@max-wittig

max-wittig commented Oct 22, 2019

Copy link
Copy Markdown
Member

@bufferoverflow Thanks for the MR!

@max-wittig
max-wittig merged commit e790b1e into master Oct 22, 2019
@bufferoverflow
bufferoverflow deleted the chore/add-test-to-dist branch October 22, 2019 06:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@max-wittig max-wittig

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1.12.1 pypi dist file doesn't include gitlab/tests/json.data

2 participants

@bufferoverflow @max-wittig