Skip to content

feat(x-deprecated-reason): add default deprecation message #2022

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 3 commits into from
Jul 15, 2025

Conversation

jamietanna
Copy link
Member

  • chore: rename variable
  • chore(examples): add a deprecated field without any reason
  • feat(x-deprecated-reason): add default deprecation message

@jamietanna jamietanna requested a review from a team as a code owner July 15, 2025 10:40
Copy link

kusari-inspector bot commented Jul 15, 2025

Kusari Analysis Results

Analysis for commit: c50a3d6, performed at: 2025-07-15T10:44:34Z

@kusari-inspector rerun - Trigger a re-analysis of this PR

@kusari-inspector feedback [your message] - Send feedback to our AI and team


Recommendation

✅ PROCEED with this Pull Request

Summary

No Flagged Issues Detected

All values appear to be within acceptable risk parameters.

No pinned version dependency changes, code issues or exposed secrets detected!

Found this helpful? Give it a 👍 or 👎 reaction!

In the case that we have a field deprecated, but no reason, the
deprecation comment isn't ideal.

Let's document it now, and improve it afterwards.
In the case that we have a field deprecated, but no reason, the
deprecation comment isn't ideal.

We can improve this by adding a default message in the case of no
`x-deprecated-reason`.
@jamietanna jamietanna force-pushed the tests/preferskipoptionalpointer branch from 63821a9 to c50a3d6 Compare July 15, 2025 10:44
Copy link

Kusari PR Analysis rerun based on - c50a3d6 performed at: 2025-07-15T10:44:34Z - link to updated analysis

@jamietanna jamietanna merged commit 7688c8d into main Jul 15, 2025
35 checks passed
@jamietanna jamietanna deleted the tests/preferskipoptionalpointer branch July 15, 2025 10:48
@jamietanna jamietanna added the enhancement New feature or request label Jul 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant