Skip to content

PR: [issue: 672] add __version__ #684

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 6 commits into from
Sep 12, 2023
Merged

PR: [issue: 672] add __version__ #684

merged 6 commits into from
Sep 12, 2023

Conversation

janvdp
Copy link
Contributor

@janvdp janvdp commented Sep 10, 2023

This a PR for #672 adding the __version__.

This PR includes the features mentioned in the issue, but no changes have been made to support poetry automatically using the llama_cpp/version.py version.

If you merge this PR as is, you still need to update llama_cpp/version.py as well as pyproject.toml.

Feel free to already merge, or provide feedback as to first adding poetry support.

@abetlen
Copy link
Owner

abetlen commented Sep 12, 2023

@janvdp thank you, looks good to me! I think you can also specify dynamic = ["version"] in the pyproject to pull it in from setuptools but I'll merge this for now.

@abetlen abetlen merged commit 6add80c into abetlen:main Sep 12, 2023
@janvdp janvdp mentioned this pull request Sep 14, 2023
@janvdp
Copy link
Contributor Author

janvdp commented Sep 14, 2023

@abetlen Thanks for the pyproject feedback. In addition to setting the "version" to dynamic, the setuptools doc state that you need to read the version: https://setuptools.pypa.io/en/latest/userguide/pyproject_config.html

I have created a PR for this: #713

I don't have a way to test it, so let me know if that it works for you.

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.

2 participants