3 Commits (08f21959ce02845f83954519c80ba0213ad7baf2)

Author	SHA1	Message	Date
Sam Blackshear	ebfe8d1e72	[litho] move reporting of missing required props out of transfer functions ... Summary: This lets us fix the limitation of reporting false positives when a `private` function calls `build()` on a parameter without passing all of the required props. We will now report such issues in the caller only if it fails to pass the required props. An unfortunate consequence of this change is that we lose track of where the actual call to `build` occurs--we now report on the declaration of the caller function rather than on the call site of `build`. I'll work on addressing that in a follow-up. Reviewed By: jeremydubreil Differential Revision: D6764153 fbshipit-source-id: 3b173e5	7 years ago
Sam Blackshear	2c6fb16636	[litho] support optional Prop's ... Summary: Don't want to warn when `Prop(optional = true)`'s aren't provided to the `Builder`. Reviewed By: jeremydubreil Differential Revision: D6741579 fbshipit-source-id: b1ae3ed	7 years ago
Sam Blackshear	4545f36875	[litho] prototype of required `@Prop` checker ... Summary: At each call to `Component$Builder.build()`, checks that the required props for `Component` have been set via prior calls to the `Builder`. Does not yet handle `Prop(optional = true)`, but will address that in a follow-up. Reviewed By: jeremydubreil Differential Revision: D6735524 fbshipit-source-id: 0c812fd	7 years ago