-
Notifications
You must be signed in to change notification settings - Fork 41
Moved the metadata into setup.cfg
.
#47
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
base: master
Are you sure you want to change the base?
Conversation
Thanks for your contribution!
Can you please explain how that's supposed to work? The tests are failing due to the missing (not yet generated?) |
e37c07f
to
4c6feea
Compare
You used to have version hardcoded into package, because fetching it using I have added it. Also I saw that the test for The decrease in coverage is because the code that has not been tested (but that has been executed) is no longer executed, the tests succeed though. |
Added `pyproject.toml`. Version is now fetched and populated automatically from git tags using `setuptools_scm`. Metadata stored in source files is fetched using `read_version`. Got rid of raw scripts, using `console_scripts` entry point since now. Dropped 2.7 out of Travis CI due to `File "tests.py", line 334, in <module> m = __import__(module, fromlist=[module]) TypeError: Item in "from list'' must be str, not unicode` error.
I've opened the discussion about dropping Python 2.7 support at #48. |
@@ -1,4 +1,5 @@ | |||
*.pyc | |||
/jsonpointer/version.py |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I know this pull request has been sitting for a while, but any chance you can elaborate on why that file would be in .gitignore
? For me it would seem logical to have it tracked.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR refactors the project’s packaging and metadata configuration by removing setup.py, moving metadata into setup.cfg, and adding pyproject.toml to automate version management with setuptools_scm. Key changes include:
- Removal of setup.py in favor of centralized configuration in setup.cfg and pyproject.toml.
- Introduction of setuptools_scm for automatic version management from git tags.
- Updates to the CI configuration in .travis.yml to utilize the new build process.
Reviewed Changes
Copilot reviewed 8 out of 8 changed files in this pull request and generated 1 comment.
Show a summary per file
File | Description |
---|---|
setup.py | Removed; metadata and script configuration moved to setup.cfg and pyproject.toml. |
setup.cfg | Added metadata, package options, and console_scripts entry point configuration. |
requirements-dev.txt | Added dependencies for build, setuptools_scm, and read_version; slight redundancy noticed. |
pyproject.toml | Introduced build system configuration supporting setuptools_scm and read_version. |
jsonpointer/init.py | Updated metadata handling; version now imported from a generated module. |
.travis.yml | Modified CI configuration to use the new build command and updated dependency installs. |
@@ -1,3 +1,7 @@ | |||
wheel | |||
twine>=1.11.0 | |||
setuptools>=38.6.0 | |||
build | |||
setuptools |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The inclusion of 'setuptools' without a version conflicts with the earlier 'setuptools>=38.6.0'. Consider consolidating these requirements to avoid potential version mismatches.
setuptools |
Copilot uses AI. Check for mistakes.
Added
pyproject.toml
.Version is now fetched and populated automatically from git tags using
setuptools_scm
.Metadata stored in source files is fetched using
read_version
.Got rid of raw scripts, using
console_scripts
entry point since now.