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

Remove semicolon from text/plain content type in example Swagger specs #2022

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

chrisinmtown
Copy link
Contributor

The extra semicolon causes a NonConformingResponseHeaders exception
when response validation is enabled for the Swagger spec in these examples.

@chrisinmtown chrisinmtown force-pushed the examples-swagger-semicolon branch from 8034736 to e9337d7 Compare December 23, 2024 21:19
@chrisinmtown chrisinmtown changed the title Remove semicolon from "text/plain" content type in Swagger specs Remove semicolon from text/plain content type in example Swagger specs Dec 23, 2024
@coveralls
Copy link

coveralls commented Dec 23, 2024

Coverage Status

coverage: 94.419%. remained the same
when pulling 03ebdb5 on chrisinmtown:examples-swagger-semicolon
into 1844a2f on spec-first:main.

The extra semicolon causes a NonConformingResponseHeaders exception
when response validation is enabled for the example Swagger specs.
@chrisinmtown chrisinmtown force-pushed the examples-swagger-semicolon branch from e9337d7 to 03ebdb5 Compare December 27, 2024 16:09
@chrisinmtown
Copy link
Contributor Author

Happy new year to the Connexion community. I think this just might be the smallest PR I have ever created ;) Please review when you can, 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.

2 participants