Skip to content

Add files in root directory to dist#369

Merged
mbohlool merged 1 commit intokubernetes-client:masterfrom
dims:add-file-to-dist-tgz
Oct 19, 2017
Merged

Add files in root directory to dist#369
mbohlool merged 1 commit intokubernetes-client:masterfrom
dims:add-file-to-dist-tgz

Conversation

@dims
Copy link
Copy Markdown
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
MANIFEST.in Outdated
exclude .gitignore
exclude .gitreview

global-exclude *.pyc No newline at end of file
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Collaborator Author

dims commented Oct 19, 2017

Done @mbohlool

bh0717-dev pushed a commit to bh0717-dev/python-dapp that referenced this pull request Apr 1, 2024
sdevtop100-dot added a commit to sdevtop100-dot/python-kubernates that referenced this pull request Feb 4, 2026
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