Skip to content

feat: use http status description as default in openapi-generator #527

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 1 commit into from
Aug 30, 2023
Merged

feat: use http status description as default in openapi-generator #527

merged 1 commit into from
Aug 30, 2023

Conversation

@bitgopatmcl
Copy link
Contributor

@bitgopatmcl bitgopatmcl commented Aug 29, 2023 •
edited
Loading

The description field is required for responses in OpenAPI. This should be a more reasonable default than the empty string.

@bitgopatmcl bitgopatmcl requested a review from a team as a code owner August 29, 2023 18:15
@andrew-scott-fischer andrew-scott-fischer merged commit ee9d522 into BitGo:master Aug 30, 2023
@github-actions
Copy link

github-actions bot commented Aug 30, 2023

🎉 This PR is included in version @api-ts/openapi-generator@2.3.0 🎉

The release is available on npm package (@latest dist-tag)

Your semantic-release bot 📦🚀

@github-actions
Copy link

github-actions bot commented Sep 26, 2023

🎉 This PR is included in version @api-ts/typed-express-router@1.1.1 🎉

The release is available on npm package (@latest dist-tag)

Your semantic-release bot 📦🚀

@github-actions
Copy link

github-actions bot commented Sep 26, 2023

🎉 This PR is included in version @api-ts/express-wrapper@1.0.21 🎉

The release is available on npm package (@latest dist-tag)

Your semantic-release bot 📦🚀

@github-actions
Copy link

github-actions bot commented Sep 26, 2023

🎉 This PR is included in version @api-ts/superagent-wrapper@1.1.12 🎉

The release is available on npm package (@latest dist-tag)

Your semantic-release bot 📦🚀

@github-actions
Copy link

github-actions bot commented Oct 4, 2023

🎉 This PR is included in version @api-ts/io-ts-http@2.4.1-beta.1 🎉

The release is available on @api-ts/io-ts-http

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@andrew-scott-fischer andrew-scott-fischer andrew-scott-fischer approved these changes

Assignees

No one assigned

Labels

released on @beta released

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@bitgopatmcl @andrew-scott-fischer