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

docs: Update README image to absolute URL, fix PyPI rendering #561

Merged
merged 2 commits into from
Jun 1, 2022

Conversation

glasnt
Copy link
Contributor

@glasnt glasnt commented May 27, 2022

When viewing the v3.1.1 README on PyPI, the new "Guide Me" button is a broken image link.

broken image

This is a known issue where PyPI needs absolute image links (ref pypi/warehouse#5582 (comment)).

By using the GitHub README's absolute URL of the image (from the raw.githubusercontent.com domain), the absolute URL can be referenced and rendered on both PyPI and GitHub README's.

Since confirming this fix requires publishing to PyPI, I've published an example package to show the before/after:

@glasnt glasnt requested review from a team as code owners May 27, 2022 04:46
@product-auto-label product-auto-label bot added size: xs Pull request size is extra small. api: logging Issues related to the googleapis/python-logging API. labels May 27, 2022
Copy link
Contributor

@arbrown arbrown 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 catching that! PyPi has not been playing nice with that addition to the readme, and now we know why!

@arbrown arbrown changed the title Update README image to absolute URL, fix PyPI rendering docs: Update README image to absolute URL, fix PyPI rendering Jun 1, 2022
@arbrown arbrown merged commit 76413b1 into googleapis:main Jun 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: logging Issues related to the googleapis/python-logging API. size: xs Pull request size is extra small.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants