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] Images links updated #72

Merged
merged 1 commit into from Nov 13, 2023
Merged

Conversation

MKhalusova
Copy link
Contributor

For consistency with other repos, doc images have been uploaded to https://huggingface.co/datasets/huggingface/documentation-images/. Thus, links on index.md have been updated.

@HuggingFaceDocBuilderDev

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

@coyotte508
Copy link
Member

The images in https://github.com/huggingface/text-embeddings-inference/tree/main/assets should be deleted? Or are they used in other places?

@MKhalusova
Copy link
Contributor Author

The images in https://github.com/huggingface/text-embeddings-inference/tree/main/assets should be deleted? Or are they used in other places?

They are also used in readme. So I left them untouched there. They may also be used somewhere else that I am not aware of, so I'm hesitant to remove them.

@MKhalusova
Copy link
Contributor Author

Gently pinging @OlivierDehaene as I don't have permissions to merge.

@OlivierDehaene OlivierDehaene merged commit 618076e into huggingface:main Nov 13, 2023
1 check passed
@OlivierDehaene
Copy link
Member

Thanks!

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