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

Don't test format parameter #523

Merged
merged 1 commit into from
Apr 14, 2022
Merged

Conversation

Azquelt
Copy link
Member

@Azquelt Azquelt commented Apr 11, 2022

Remove PetStoreAppTest.testJsonResponseTypeWithQueryParameter because
support for the format parameter is optional.

Fixes #503

Remove PetStoreAppTest.testJsonResponseTypeWithQueryParameter because
support for the format parameter is optional.
Copy link
Member

@MikeEdgar MikeEdgar left a comment

Choose a reason for hiding this comment

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

LGTM

@Azquelt Azquelt merged commit f537bfc into eclipse:master Apr 14, 2022
@Azquelt Azquelt deleted the optional-format branch April 14, 2022 10:01
@MikeEdgar MikeEdgar added this to the MP OpenAPI 3.1 milestone Apr 28, 2022
@Azquelt Azquelt removed this from the MP OpenAPI 3.1 milestone Jun 13, 2022
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.

Support for query parameter format is optional according to spec but is required by the TCK
2 participants