-
Notifications
You must be signed in to change notification settings - Fork 4.7k
docs (#18): add deployment page #412
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
Conversation
7d45ef0
to
958d79a
Compare
@phanan @NataliaTepluhina - Would love your review when you have a moment! |
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 isn't a subject I know much about but I've provided what little feedback I can.
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.
Minor changes here and there, but really close! @skirtles-code gave a great foundation of the review here
Great job @bencodezen and @skirtles-code! |
* docs (#18): add deployment page * docs: remove named file * docs: fix app call * Update src/guide/tooling/deployment.md * Update src/guide/tooling/deployment.md * Update src/guide/tooling/deployment.md * Update src/guide/tooling/deployment.md * Update src/guide/tooling/deployment.md * Update src/guide/tooling/deployment.md * Update src/guide/tooling/deployment.md * remove Webpack 3 details Co-authored-by: Sarah Drasner <[email protected]>
This is mostly a copy and paste from the v2 docs. As far as I'm aware, I don't think anything major has changed CLI wise from v2 to v3. Let me know if I missed anything @sodatea!
P.S. Feel free to add any other reviewers who might know more about this section.
Netlify Preview