Skip to content

When using @Operation, treat deprecated false as NULL (reduce openap.json noise) #575

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

Merged
merged 2 commits into from
Mar 10, 2025

Conversation

rbygrave
Copy link
Contributor

The prior change adding reading @operation, introduced an issue in that all non-deprecated methods then explicitly had in openapi.json a deprecated = false attribute.

This "fixes" that issue by only setting deprecated attribute when it is true.

….json noise)

The prior change adding reading @operation, introduced an issue in that all non-deprecated methods then explicitly had in openapi.json a deprecated = false attribute.

This "fixes" that issue by only setting deprecated attribute when it is true.
@rbygrave rbygrave requested a review from SentryMan March 10, 2025 03:45
@rbygrave rbygrave self-assigned this Mar 10, 2025
@rbygrave rbygrave merged commit 3ed55e5 into master Mar 10, 2025
7 checks passed
@rbygrave rbygrave deleted the feature/openapi-deprecated branch March 10, 2025 07:02
@SentryMan SentryMan added this to the 3.1 milestone Mar 15, 2025
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