Skip to content
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

Improve packaging metadata for PyPI #28

Closed
goodmami opened this issue Feb 8, 2023 · 4 comments
Closed

Improve packaging metadata for PyPI #28

goodmami opened this issue Feb 8, 2023 · 4 comments

Comments

@goodmami
Copy link

goodmami commented Feb 8, 2023

The tiktoken package on PyPI could use some metadata to indicate that it is an official OpenAI project because, unlike this repo, the project on PyPI does not mention its link to OpenAI at all.

  • Add the openai user as an author or maintainer
  • Link back to this repo as the project homepage
  • Use the README as a project description

These steps help because phishing and other security issues are a problem on PyPI, and linking back to this OpenAI-controlled repo (which then indicates the proper package in PyPI to install) provide a signal that it's a legit package, at least for those of us who double-check package sources before pip installing :)

@zihaolucky
Copy link

Another question, is it possible to support py36?

@hauntsaninja
Copy link
Collaborator

goodmami, thanks for the suggestion, will make sure to include more metadata in the next release

@hauntsaninja
Copy link
Collaborator

hauntsaninja commented Feb 13, 2023

zihaolucky, that's unrelated to this issue. As stated in #1 (comment) , I will not be adding support for Python 3.7 or older. Python 3.6 has been end of life for over a year, so I strongly recommend against using it in production.

@hauntsaninja
Copy link
Collaborator

Fixed in 0.3.0. Thanks for the suggestion! :-)

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

No branches or pull requests

3 participants