-
Notifications
You must be signed in to change notification settings - Fork 160
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
Add long description to setup.py #1094
Add long description to setup.py #1094
Conversation
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.
Thank you. Could you pls run python setup.py bdist_wheel
on your local env to see the result?
@ejguan has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
Validated on my local environment after the merge |
Summary: Fixes #1088 ### Changes - Added the README.md as a long description in the `setup.py` - Added `long_description_content_type` field Co-Authored by: Martin Achtner <[email protected]> Jad Saliba <[email protected]> Pull Request resolved: #1094 Reviewed By: NivekT Differential Revision: D44166989 Pulled By: ejguan fbshipit-source-id: 24fafc170a825372356aba8c3f086a360902b6c9
Summary: Fixes #1088 ### Changes - Added the README.md as a long description in the `setup.py` - Added `long_description_content_type` field Co-Authored by: Martin Achtner <[email protected]> Jad Saliba <[email protected]> Pull Request resolved: #1094 Reviewed By: NivekT Differential Revision: D44166989 Pulled By: ejguan fbshipit-source-id: 24fafc170a825372356aba8c3f086a360902b6c9
Fixes #1088
Changes
setup.py
long_description_content_type
fieldCo-Authored by:
Martin Achtner [email protected]
Jad Saliba [email protected]