Skip to content
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

issue #222 #234

Merged
merged 36 commits into from May 3, 2019
Merged

issue #222 #234

Changes from 2 commits
Commits
Show all changes
36 commits
Select commit Hold shift + click to select a range
062ffb5
Added test case for issue 222
Apr 9, 2019
08f9932
It finally compiles! Except for java :(
Apr 12, 2019
2ebdf07
Merge branch 'master' into issue222
tomasherman Apr 12, 2019
28b1b6b
Added test case for issue 222
Apr 25, 2019
cc1cc1b
Merge branch 'issue222' of github.com:avast/guardrail into issue222
Apr 25, 2019
184a5b9
Test passes
Apr 25, 2019
e014a04
Merge branch 'master' into issue222
tomasherman Apr 25, 2019
61d15f2
Reimplemented
Apr 25, 2019
5cd28fb
Merge branch 'issue222' of github.com:avast/guardrail into issue222
Apr 25, 2019
ee39921
Attempt to fix jacksonˆ
Apr 25, 2019
7a9b6da
Removed unused values
Apr 25, 2019
e525c60
Scan all parents for discriminators
Apr 25, 2019
db374e4
Cleanup
Apr 26, 2019
89cc0c4
Update modules/codegen/src/main/scala/com/twilio/guardrail/generators…
blast-hardcheese Apr 26, 2019
d2e7b96
Code review improvements #2
Apr 26, 2019
f244636
Merge branch 'master' into issue222
tomasherman Apr 26, 2019
cf0ddf9
Merge branch 'master' into issue222
tomasherman Apr 27, 2019
8c636a8
Removed comment
Apr 27, 2019
c3a8d6f
Merge branch 'issue222' of github.com:avast/guardrail into issue222
Apr 27, 2019
8411ede
Merge branch 'master' into issue222
tomasherman Apr 30, 2019
0efb82e
Merge branch 'master' of https://github.com/twilio/guardrail into iss…
May 1, 2019
e97efbd
Added another test case and implementation
May 1, 2019
0d54cab
Improved 2nd test case scenario
May 1, 2019
2a7e2fc
Merge branch 'issue222' of github.com:avast/guardrail into issue222
May 1, 2019
8aec398
Added another test case
May 1, 2019
83a5cfc
Added better error reporting when resolving references
May 1, 2019
bb38c38
Scalafmt
May 1, 2019
cb96779
Attempt to reduce filename sizes
May 1, 2019
f81912b
Update modules/codegen/src/main/scala/com/twilio/guardrail/generators…
blast-hardcheese May 2, 2019
5a75f7a
Merge branch 'master' of https://github.com/twilio/guardrail into iss…
May 2, 2019
20ad1f6
Merge branch 'issue222' of github.com:avast/guardrail into issue222
May 2, 2019
6c37f8a
Merge branch 'master' into issue222
tomasherman May 2, 2019
c0b3ddd
Merge branch 'master' into issue222
tomasherman May 2, 2019
fcbddde
Reverted changes to jackson generation
May 2, 2019
b5bfc35
Merge branch 'master' into issue222
tomasherman May 3, 2019
aa2ba14
Merge branch 'master' into issue222
blast-hardcheese May 3, 2019
File filter...
Filter file types
Jump to…
Jump to file or symbol
Failed to load files and symbols.

Always

Just for now

@@ -105,7 +105,7 @@ object CirceProtocolGenerator {
case m: ObjectSchema => Target.pure(Option(m.getProperties))
case comp: ComposedSchema =>
val extractedProps =
Option(comp.getAllOf()).toList.flatMap(_.asScala.toList).map(e => Option(e.getProperties).map(_.asScala.toMap)).collect { case Some(e) => e }
Option(comp.getAllOf()).toList.flatMap(_.asScala.toList).flatMap(e => Option(e.getProperties).map(_.asScala.toMap))
val mergedProps = extractedProps.fold(Map.empty)(_ ++ _)
Target.pure(Option(mergedProps.asJava))
case comp: Schema[_] if Option(comp.get$ref).isDefined =>
ProTip! Use n and p to navigate between commits in a pull request.
You can’t perform that action at this time.