Skip to content

Jsx#128

Merged
carburo merged 39 commits into
reactjs:masterfrom
jvelezpo:jsx
Feb 12, 2019
Merged

Jsx#128
carburo merged 39 commits into
reactjs:masterfrom
jvelezpo:jsx

Conversation

@jvelezpo

@jvelezpo jvelezpo commented Feb 9, 2019

Copy link
Copy Markdown
Contributor

Relates to: #4

@carburo carburo left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Here we go! Thank you for your contribution. I made several comments. Review them, commit the proposed changes if you agree and otherwise tell me why should we leave it as is.

Comment thread content/docs/jsx-in-depth.md Outdated
Comment thread content/docs/jsx-in-depth.md Outdated
Comment thread content/docs/jsx-in-depth.md Outdated
Comment thread content/docs/jsx-in-depth.md Outdated
Comment thread content/docs/jsx-in-depth.md Outdated
Comment thread content/docs/jsx-in-depth.md Outdated
Comment thread content/docs/jsx-in-depth.md Outdated
Comment thread content/docs/jsx-in-depth.md Outdated
Comment thread content/docs/jsx-in-depth.md Outdated
Comment thread content/docs/jsx-in-depth.md Outdated
Comment thread content/docs/jsx-in-depth.md Outdated
Comment thread content/docs/jsx-in-depth.md Outdated
carburo and others added 22 commits February 12, 2019 08:09
Co-Authored-By: jvelezpo <jvelezpo@users.noreply.github.com>
Co-Authored-By: jvelezpo <jvelezpo@users.noreply.github.com>
Co-Authored-By: jvelezpo <jvelezpo@users.noreply.github.com>
Co-Authored-By: jvelezpo <jvelezpo@users.noreply.github.com>
Co-Authored-By: jvelezpo <jvelezpo@users.noreply.github.com>
Co-Authored-By: jvelezpo <jvelezpo@users.noreply.github.com>
Co-Authored-By: jvelezpo <jvelezpo@users.noreply.github.com>
Co-Authored-By: jvelezpo <jvelezpo@users.noreply.github.com>
Co-Authored-By: jvelezpo <jvelezpo@users.noreply.github.com>
Co-Authored-By: jvelezpo <jvelezpo@users.noreply.github.com>
Co-Authored-By: jvelezpo <jvelezpo@users.noreply.github.com>
Co-Authored-By: jvelezpo <jvelezpo@users.noreply.github.com>
Co-Authored-By: jvelezpo <jvelezpo@users.noreply.github.com>
Co-Authored-By: jvelezpo <jvelezpo@users.noreply.github.com>
Co-Authored-By: jvelezpo <jvelezpo@users.noreply.github.com>
Co-Authored-By: jvelezpo <jvelezpo@users.noreply.github.com>
Co-Authored-By: jvelezpo <jvelezpo@users.noreply.github.com>
Co-Authored-By: jvelezpo <jvelezpo@users.noreply.github.com>
Co-Authored-By: jvelezpo <jvelezpo@users.noreply.github.com>
Co-Authored-By: jvelezpo <jvelezpo@users.noreply.github.com>
Co-Authored-By: jvelezpo <jvelezpo@users.noreply.github.com>
Co-Authored-By: jvelezpo <jvelezpo@users.noreply.github.com>
carburo and others added 14 commits February 12, 2019 08:16
Co-Authored-By: jvelezpo <jvelezpo@users.noreply.github.com>
Co-Authored-By: jvelezpo <jvelezpo@users.noreply.github.com>
Co-Authored-By: jvelezpo <jvelezpo@users.noreply.github.com>
Co-Authored-By: jvelezpo <jvelezpo@users.noreply.github.com>
Co-Authored-By: jvelezpo <jvelezpo@users.noreply.github.com>
Co-Authored-By: jvelezpo <jvelezpo@users.noreply.github.com>
Co-Authored-By: jvelezpo <jvelezpo@users.noreply.github.com>
Co-Authored-By: jvelezpo <jvelezpo@users.noreply.github.com>
Co-Authored-By: jvelezpo <jvelezpo@users.noreply.github.com>
Co-Authored-By: jvelezpo <jvelezpo@users.noreply.github.com>
Co-Authored-By: jvelezpo <jvelezpo@users.noreply.github.com>
Co-Authored-By: jvelezpo <jvelezpo@users.noreply.github.com>
Co-Authored-By: jvelezpo <jvelezpo@users.noreply.github.com>
Co-Authored-By: jvelezpo <jvelezpo@users.noreply.github.com>
@carburo

carburo commented Feb 12, 2019

Copy link
Copy Markdown
Member

Ok, thank you! Merging!

@carburo carburo merged commit 4d9afb7 into reactjs:master Feb 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants