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.

Sign up for GitHub

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

Jump to bottom

Documentation for null initialization of initialVariables #701

Closed
wants to merge 1 commit into from
Closed

Documentation for null initialization of initialVariables #701

wants to merge 1 commit into from

Conversation

RavenHursT
Copy link

Added information in Container API docs for null usage in initialVariables. Fixes #700

@facebook-github-bot
Copy link
Contributor

Thank you for your pull request and welcome to our community. We require contributors to sign our Contributor License Agreement, and we don't seem to have you on file. In order for us to review and merge your code, please sign up at https://code.facebook.com/cla - and if you have received this in error or have any questions, please drop us a line at cla@fb.com. Thanks!

@facebook-github-bot
Copy link
Contributor

Thank you for signing our Contributor License Agreement. We can now accept your code for this (and any) Facebook open source project. Thanks!

@yungsters
Copy link
Contributor

Thanks for documenting these. It's not great that these are necessary, but it's better to have the documented than not.

@yungsters
Copy link
Contributor

@facebook-github-bot shipit

@facebook-github-bot
Copy link
Contributor

Thanks for importing. If you are an FB employee go to Phabricator to review.

@ghost ghost closed this in 9ec5d79 Mar 3, 2016
venepe pushed a commit to venepe/relay that referenced this pull request Mar 7, 2016
@venepe
Documentation for null initialization of initialVariables
e46d37c 
Summary:Added information in Container API docs for null usage in initialVariables. Fixes facebook#700
Closes facebook#701

Differential Revision: D3001979

Pulled By: yungsters

fb-gh-sync-id: ceaf7e4cf0f45ed4f354a041b6ab474b66d23eff
shipit-source-id: ceaf7e4cf0f45ed4f354a041b6ab474b66d23eff
@RavenHursT
Copy link
Author

Thanks @yungsters

This was referenced May 25, 2021
Open
Open
Open
Open
@joseroubert08 joseroubert08 mentioned this pull request Nov 28, 2023
Open
@enterstudio enterstudio mentioned this pull request Nov 30, 2023
Open
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
CLA Signed GH Review: accepted
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@RavenHursT @facebook-github-bot @yungsters