Consider making `isInliningPropertiesFromReferencedSchemas` the default behaviour about createapi HOT 3 CLOSED

not sure if the Pet even needs to be generated
generate Pet as a protocol

I've just started reading OpenAPI to learn but it seems more safe to inline or protocol as an inheritance hierarchy cannot necessarily be assumed with allOf.

I can think of cases where people want either of these generation options. We would have to make sure that entity generation doesn't follow isMakingClassesFinal.

# Apply to all objects
# default: inline
parentObjectGeneration: entity || protocol || inline

# Specify per type
# default: inline
parentObjectGeneration:
  - Pet: protocol
  - Human: entity

Edit: don't do that. We should think about how discriminators would create inheritances but I think only with protocols. https://swagger.io/docs/specification/data-models/inheritance-and-polymorphism/

from createapi.

I think it's a good idea to make this behavior the default. Looking at the example, if this I'm not sure if the Pet even needs to be generated in this case (assuming it's only ever used in allOf and never referenced as a separate entity).

from createapi.

I would suggest keeping this option because there are generally three types of ways to address it: composition, inheritance, and inlining all properties. CreateAPI covers everything but inheritance right now.

One more thing CreateAPI could do when inlining is used, is generate Pet as a protocol and add conformance to it to the types like Dog.

from createapi.