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

To fix a few formatting typos in Gradio-and-Wandb-Integration.md tutorial #8563

Merged
merged 3 commits into from
Jun 20, 2024

Conversation

sadra-barikbin
Copy link
Contributor

Description

Hi there!

There was some formatting typos in the tutorial. This PR fixes them.

@gradio-pr-bot
Copy link
Contributor

gradio-pr-bot commented Jun 16, 2024

🪼 branch checks and previews

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

@abidlabs abidlabs requested a review from aliabd June 19, 2024 15:15
@gradio-pr-bot
Copy link
Contributor

gradio-pr-bot commented Jun 19, 2024

🦄 no changes detected

This Pull Request does not include changes to any packages.

__No changes detected. __

  • Maintainers can select this checkbox to manually select packages to update.

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 create the changelog file directly.

Copy link
Collaborator

@aliabd aliabd left a comment

Choose a reason for hiding this comment

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

Nice, thanks @sadra-barikbin!

There are still some issues that are breaking the rendering. So i left some suggestions. But there is one important one. For some reason double empty lines can cause the codeblock to misbehave. Can you remove the following lines?

By the way, you can see a preview of the guide by visiting the website link at the gradio-pr-bot comment, just make sure to toggle to 'main' on the guide page to see the correct version.

Co-authored-by: Ali Abdalla <ali.abdalla@huggingface.co>
@pngwn pngwn enabled auto-merge (squash) June 20, 2024 08:40
@pngwn pngwn merged commit 4c8a183 into gradio-app:main Jun 20, 2024
7 checks passed
Copy link
Member

@pngwn pngwn left a comment

Choose a reason for hiding this comment

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

I've applied @aliabd's suggestions and will merge when CI runs. Thanks so much @sadra-barikbin!

@sadra-barikbin sadra-barikbin deleted the patch-1 branch June 21, 2024 15:28
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.

None yet

4 participants