Skip to content

Conversation

aliabd
Copy link
Collaborator

@aliabd aliabd commented Aug 25, 2025

#11773 (comment)

example:
Screenshot 2025-08-25 at 11 40 30 PM

@aliabd aliabd requested a review from abidlabs August 25, 2025 20:50
@gradio-pr-bot
Copy link
Collaborator

gradio-pr-bot commented Aug 25, 2025

🪼 branch checks and previews

Name Status URL
Spaces ready! Spaces preview
Website ready! Website preview
Storybook ready! Storybook preview
🦄 Changes detected! Details

Install Gradio from this PR

pip install https://gradio-pypi-previews.s3.amazonaws.com/11b0deb195f960127de2ac242380ac7f94514a80/gradio-5.43.1-py3-none-any.whl

Install Gradio Python Client from this PR

pip install "gradio-client @ git+https://github.com/gradio-app/gradio@11b0deb195f960127de2ac242380ac7f94514a80#subdirectory=client/python"

Install Gradio JS Client from this PR

npm install https://gradio-npm-previews.s3.amazonaws.com/11b0deb195f960127de2ac242380ac7f94514a80/gradio-client-1.17.1.tgz

Use Lite from this PR

<script type="module" src="https://gradio-lite-previews.s3.amazonaws.com/11b0deb195f960127de2ac242380ac7f94514a80/dist/lite.js""></script>

@gradio-pr-bot
Copy link
Collaborator

gradio-pr-bot commented Aug 25, 2025

🦄 change detected

This Pull Request includes changes to the following packages.

Package Version
gradio minor
website minor

  • Fix markdown link rendering in docs descriptions

‼️ Changeset not approved by maintainers. Ensure the version bump is appropriate for all packages before approving.

  • Maintainers can approve the changeset by selecting this checkbox.

Something isn't right?

  • Maintainers can change the version label to modify the version bump.
  • If the bot has failed to detect any changes, or if this pull request needs to update multiple packages to different versions or requires a more comprehensive changelog entry, maintainers can update the changelog file directly.

@abidlabs
Copy link
Member

Actually, would it be possible to just autorecognize links (any single word that begins with https:// or http://) and render that as a link? That way, we could keep the html/markdown syntax out of code docstrings and revert #11773

@aliabd
Copy link
Collaborator Author

aliabd commented Aug 25, 2025

yeah @abidlabs good point not a good idea to have this in the docstrings

@aliabd
Copy link
Collaborator Author

aliabd commented Aug 25, 2025

@abidlabs done and reverted the login button docstring

Copy link
Member

@abidlabs abidlabs left a comment

Choose a reason for hiding this comment

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

lgtm!

@abidlabs abidlabs merged commit 5a4ae49 into main Aug 26, 2025
22 of 23 checks passed
@abidlabs abidlabs deleted the markdown-desc branch August 26, 2025 13:11
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.

3 participants