mulesoft-labs / raml-java-tools

Apache License 2.0
19 stars 20 forks source link

Union fix #83

Open Kaiser1989 opened 3 years ago

Kaiser1989 commented 3 years ago

Although i added some union implementations before, there were still bugs with union handling.

Following a raml example with all upper problems in one:

one:
  type: object
  properties:
    mem1: string | nil
    mem2: string | nil

second:
  type: one
  properties:
    mem3: number[] | string[] | nil

This Pull Request fixes all of these bugs. But there are some breaking changes:

I don't know, if this project is still alive, but we needed these features and I implemented them for us. As we gain lots of profit from this project, I want to share my results. It's up to you to accept them.

Best regards Philipp

jstoiko commented 3 years ago

Thank you for your contribution @Kaiser1989. Unfortunately this project is no longer maintained and should have been archived. I will notify the owners. This project originally spawned out of RAML for JAX-RS which has been archived. Please feel free to reach out if you need assistance/guidance in maintaining a fork.