Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Add files in root directory to dist #369

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

Merged
merged 1 commit into from
Oct 19, 2017

Conversation

dims
Copy link
Collaborator

@dims dims commented Oct 18, 2017

By adding a MANIFEST.in, we can list the files we need to include
into the tgz when we run "python setup.py sdist". With this change
you should see the following files in the tgz file.

LICENSE
MANIFEST.in
OWNERS
PKG-INFO
README.md
RELEASE.md
requirements.txt
setup.cfg
setup.py
test-requirements.txt
tox.ini

Fixes #361

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels Oct 18, 2017
exclude .gitignore
exclude .gitreview

global-exclude *.pyc
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: add new line at the end of the file.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done

@mbohlool
Copy link
Contributor

Thanks for the fix @dims. Just one nit otherwise LGTM.

@dims dims force-pushed the add-file-to-dist-tgz branch from 19fb5ad to 480723f Compare October 18, 2017 22:42
By adding a MANIFEST.in, we can list the files we need to include
into the tgz when we run "python setup.py sdist". With this change
you should see the following files in the tgz file.

LICENSE
MANIFEST.in
OWNERS
PKG-INFO
README.md
RELEASE.md
requirements.txt
setup.cfg
setup.py
test-requirements.txt
tox.ini
@dims dims mentioned this pull request Oct 19, 2017
@mbohlool mbohlool merged commit de1d763 into kubernetes-client:master Oct 19, 2017
@mbohlool
Copy link
Contributor

@dims, I merged this but can you also add a line to the CHANGELOG for this?

dims added a commit to dims/client-python that referenced this pull request Oct 19, 2017
@dims dims mentioned this pull request Oct 19, 2017
@dims
Copy link
Collaborator Author

dims commented Oct 19, 2017

Done @mbohlool

bh717 pushed a commit to bh717/python-dapp that referenced this pull request Apr 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/XS Denotes a PR that changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants