Skip to content

Add initial skeleton files - #1

Merged
steven-esser merged 9 commits into
developfrom
add-initial-skeleton-files
Oct 22, 2020
Merged

steven-esser merged 9 commits into
developfrom
add-initial-skeleton-files

Conversation

@steven-esser

Copy link
Copy Markdown
Contributor

This commit adds the inital skeleton files needed for a bare-bones
python library. Includes a simple configure script and setup.py file

Signed-off-by: Steven Esser sesser@nexb.com

This commit adds the inital skeleton files needed for a bare-bones
python library. Includes a simple configure script and setup.py file

Signed-off-by: Steven Esser <sesser@nexb.com>
@steven-esser

Copy link
Copy Markdown
Contributor Author

This a is simple base to get started with breaking out various python libraries. We can keep this updated as we develop a concrete set of conventions. Following this as a guide: https://blog.jaraco.com/a-project-skeleton-for-python-projects/ we will be able to update our already existing repos + all new repos with this skeleton structure.

Comment thread requirements-tests.txt Outdated

@JonoYang JonoYang left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

lgtm. The only change I can think of is to add pytest-xdist to requirements-tests.txt just so we can run multiple tests at once, but that is minor.

Signed-off-by: Steven Esser <sesser@nexb.com>
* Add AUTHORS.rst
* Add CHANGELOG.rst
* Add setup.cfg

Signed-off-by: Steven Esser <sesser@nexb.com>
* Add PEP 517/518 pyproject.toml file
* Add setuptools_scm to handle versioning
* Add setup.py content to setup.cfg
* Update setup.py to act as a shim (so pip install -e works)

Addresses: #2

Signed-off-by: Steven Esser <sesser@nexb.com>
Signed-off-by: Steven Esser <sesser@nexb.com>

@JonoYang JonoYang left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I am able to merge in the changes into plugincode, run the configure script, and make a wheel. I noticed that the version number of the wheel was 0.1.dev102+gc6396ae.d20200925. Can I set the version number to start from 20.09 in setup.cfg?

@steven-esser

steven-esser commented Sep 25, 2020

Copy link
Copy Markdown
Contributor Author

@JonoYang The version is now based of the git tag at the time of wheel creation. Try checking out to v20.09 tag and creating the wheel there.

@steven-esser

steven-esser commented Sep 25, 2020

Copy link
Copy Markdown
Contributor Author

@JonoYang Actually, you will need to create a v20.09 tag first, and it should work (there are no tags right now in plugincode)

See https://github.com/pypa/setuptools_scm#default-versioning-scheme for more details.

@pombredanne pombredanne left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looking good! See a comment inline.

Comment thread CHANGELOG.rst
@@ -0,0 +1,5 @@
Release notes
-------------
### Version 0.0.0

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

What's the best way to generate this? I wonder if adopting towncrier would be a good thing?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

#9 opened for this. This will require a bit more work, so I will do that in a different PR/branch

Signed-off-by: Steven Esser <sesser@nexb.com>
Signed-off-by: Steven Esser <sesser@nexb.com>
@steven-esser
steven-esser force-pushed the add-initial-skeleton-files branch from fcf8789 to 343ff29 Compare September 30, 2020 19:17
Signed-off-by: Steven Esser <sesser@nexb.com>
Comment thread setup.cfg Outdated
Signed-off-by: Steven Esser <sesser@nexb.com>
@steven-esser
steven-esser merged commit ad02f4e into develop Oct 22, 2020
@steven-esser
steven-esser deleted the add-initial-skeleton-files branch October 22, 2020 20:04
JonoYang pushed a commit that referenced this pull request Sep 7, 2021
JonoYang pushed a commit that referenced this pull request Sep 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants