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

Add yaml string support #84

Merged
merged 3 commits into from
Apr 18, 2022
Merged

Add yaml string support #84

merged 3 commits into from
Apr 18, 2022

Conversation

clearydude
Copy link
Contributor

Feel free to take or leave this PR but I'm working with yaml-based open API specs and figured it was an easy add.

Note: I copied the pattern and used serde_yaml as the footnote text but I think it (and maybe the json ones above it) should just read yaml (and json)

@juhaku
Copy link
Owner

juhaku commented Apr 14, 2022

Awesome. Yeah. been thinking to add it but not been high on the list since haven't had a need for that.
Could you add short documentation to the REAMDE.md and utoipa/lib.rs under Features section?

@juhaku
Copy link
Owner

juhaku commented Apr 14, 2022

And true.. should read json and yaml. feel free to fix :)

@clearydude
Copy link
Contributor Author

I'll add some documentation and update this PR (probably tomorrow). Thanks!!

Copy link
Owner

@juhaku juhaku left a comment

Choose a reason for hiding this comment

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

Sweet 🍬

@juhaku juhaku merged commit 96e0228 into juhaku:master Apr 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Released
Development

Successfully merging this pull request may close these issues.

2 participants