Skip to content

Add REST API markdown docs based on openapi definition#363

Merged
lucas-koehler merged 1 commit intomainfrom
lk/openapi-readme-generation
Oct 25, 2024
Merged

Add REST API markdown docs based on openapi definition#363
lucas-koehler merged 1 commit intomainfrom
lk/openapi-readme-generation

Conversation

@lucas-koehler
Copy link
Contributor

@lucas-koehler lucas-koehler commented Oct 24, 2024

Use the openapi-generator-cli to generate markdown docs from the openapi.json.
Also document how this works and add a hint that the generator supports
various doc and code languages.

Use the openapi-generator-cli to generate markdown docs from the openapi.json.
Also document how this works and add a hint that the generator supports
various doc and code languages.
@lucas-koehler lucas-koehler force-pushed the lk/openapi-readme-generation branch from 249612d to 8215bf1 Compare October 24, 2024 13:30
Copy link
Contributor

@jfaltermeier jfaltermeier left a comment

Choose a reason for hiding this comment

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

LGTM. Regeneration worked on my side as well.

The generated markdown files don’t follow our formatting rules, but since we shouldn’t be editing generated files manually anyway, I’m not sure if it’s an issue. What’s your take on this?

@lucas-koehler
Copy link
Contributor Author

The generated markdown files don’t follow our formatting rules, but since we shouldn’t be editing generated files manually anyway, I’m not sure if it’s an issue. What’s your take on this?

We could format them using prettier on the command line. However, I don't see any reason to do this for now because - as you stated - these files should not be edited manually anyway. Thus, I would not add a step in the documentation for now.

@lucas-koehler lucas-koehler merged commit 8215bf1 into main Oct 25, 2024
@lucas-koehler lucas-koehler deleted the lk/openapi-readme-generation branch October 25, 2024 09:38
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.

2 participants