-
-
Notifications
You must be signed in to change notification settings - Fork 8.7k
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
Add docker support #936
Add docker support #936
Conversation
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. If you are contributing on behalf of someone else (eg your employer), the individual CLA may not be sufficient and your employer may need the corporate CLA signed. If you have received this in error or have any questions, please contact us at [email protected]. Thanks! |
Deploy preview for docusaurus-preview ready! Built with commit 3094afd |
Thank you for signing our Contributor License Agreement. We can now accept your code for this (and any) Facebook open source project. Thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks a lot. Just very small nits.
lib/copy-examples.js
Outdated
)}. Creating an example gitignore file for you to copy from if desired.\n` | ||
|
||
const copyFileOutWebsiteFolder = (fileNameFrom, isHide) => { | ||
let FileNameTo = isHide ? `.${fileNameFrom}` : fileNameFrom; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
use camelCase
let fileNameTo
04babcb
to
a1a26d1
Compare
done |
b28a829
to
b00b933
Compare
I'll let another reviewer put approval stamp for docs nits/ etc on this before merging |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks good to me. 👍 I would like @ericnakagawa to have a look at this as well.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great to me!
I contributed some light copy changes to the docs.
Thank you @wszgxa for your contribution!
Motivation
Github issue: #926
Add docker support in basic example file and doc.
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
No break change, need someone to try the document. I have tested it locally.
Related PRs
nope