Skip to content
This repository was archived by the owner on Jun 22, 2025. It is now read-only.

Fix wrong arguments for URL to GitHub#209

Merged
samuelhwilliams merged 2 commits intopython-eel:masterfrom
andriyor:master
Nov 18, 2019
Merged

Fix wrong arguments for URL to GitHub#209
samuelhwilliams merged 2 commits intopython-eel:masterfrom
andriyor:master

Conversation

@andriyor
Copy link
Copy Markdown
Contributor

@andriyor andriyor commented Nov 15, 2019

Warehouse now uses the url provided to display links in the sidebar, as well as including them in API responses to help automation tool find the source code for Eel. For example, see Django's setup.cfg and PyPI listing.

PyPI also already support rendering project descriptions from Markdown https://dustingram.com/articles/2018/03/16/markdown-descriptions-on-pypi/

use full readme.md for long_description
use absolute pointing at github raw image
Comment thread setup.py
Copy link
Copy Markdown
Collaborator

@samuelhwilliams samuelhwilliams left a comment

Choose a reason for hiding this comment

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

Thanks for doing this - looks like it does indeed work now (https://test.pypi.org/project/Eel/0.11.0/), albeit I need you to make one addition as suggested below (you should be able to click to accept it).

Once that's in, I'll approve and we can merge it.

Really appreciate you taking the time to put up this pull request.

Comment thread setup.py
Co-Authored-By: Samuel Williams <samuelhwilliams@users.noreply.github.com>
@samuelhwilliams samuelhwilliams merged commit 5f62b06 into python-eel:master Nov 18, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants