Skip to content

Conversation

@coyotte508
Copy link
Member

cc @Wauplin

Any reason for the trailing | for each line in the table?

@HuggingFaceDocBuilderDev

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

@julien-c
Copy link
Member

julien-c commented Oct 3, 2022

include a screenshot :)

@coyotte508
Copy link
Member Author

@julien-c
Copy link
Member

julien-c commented Oct 3, 2022

i would refer to a detailed section below the table because this is not super legible:

image

@coyotte508
Copy link
Member Author

coyotte508 commented Oct 3, 2022

Or we could overhaul the endpoint docs and use an interactive swagger-like interface x)

There's npm modules for it 🤭 .

@NimaBoscarino
Copy link
Contributor

I don't think there's any reason for the trailing |, as far as I know. Overhauling the endpoints docs to use something like Swagger would be awesome 👍 In the meantime though I agree that the payload is too long for the table, so maybe defining the json portion under the table would be cool, like Julien suggested.

@coyotte508 coyotte508 marked this pull request as draft October 20, 2022 21:55
@osanseviero osanseviero marked this pull request as ready for review January 26, 2023 11:21
@osanseviero
Copy link
Contributor

Closing this as it's outdated/stalled

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.

6 participants