handle value types like Option[Seq[T]]

[pjfanning]

No description provided.

[codecov]

Codecov Report

Merging #174 (feb7aa3) into develop (4c1bc86) will increase coverage by 0.13%.
The diff coverage is 100.00%.

@@             Coverage Diff             @@
##           develop     #174      +/-   ##
===========================================
+ Coverage    88.20%   88.33%   +0.13%     
===========================================
  Files            3        3              
  Lines          178      180       +2     
  Branches        18       17       -1     
===========================================
+ Hits           157      159       +2     
  Misses          21       21              

Impacted Files	Coverage Δ
...github/swagger/scala/converter/ErasureHelper.scala	95.45% <100.00%> (+0.45%)	⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[pjfanning]

@SamTheisens could you have a look at this? I think it is a necessary change for handling types like Option[Seq[T]].

[edit] I later added 60dbce2 to handle deeper nesting

[SamTheisens]

@SamTheisens could you have a look at this? I think it is a necessary change for handling types like Option[Seq[T]].

[edit] I later added 60dbce2 to handle deeper nesting

This looks good to me 👍 . I tend to avoid optional sequences in APIs because their semantics aren't clear to me.
In theory I guess you could infinitely nest Option[Option[Option...., which isn't handled by the current code. But not sure fields like that belongs in any API :-)