Skip to content

Conversation

@newtork
Copy link
Contributor

@newtork newtork commented Jan 10, 2025

Context

Issue found here
SAP/ai-sdk-java#267 (review)

Fix inconsistency, where all model classes marked as @Beta with the exception of the anyof/oneof -based interfaces.

Feature scope:

We maintain a feature toggle to mark generated code as "unstable".
But we forgot to consider interface types, that were introduced afterwards.

  • Fix mustache file template
  • Update existing test case
    • Update assertions

Definition of Done

  • Functionality scope stated & covered
  • Tests cover the scope above
  • Error handling created / updated & covered by the tests above
  • Documentation updated
  • Release notes updated (The generator behavior-change only applies when the feature toggle is active)

@newtork newtork added please merge Request to merge a pull request please review Request to review a pull request labels Jan 10, 2025
/**
* OneOf
*/
@Beta
Copy link
Contributor Author

Choose a reason for hiding this comment

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

(Comment)

This annotation was not generated before.

@newtork newtork merged commit 3844e4e into main Jan 14, 2025
14 checks passed
@newtork newtork deleted the openapi/anyof-annotation-beta branch January 14, 2025 12:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

please merge Request to merge a pull request please review Request to review a pull request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants