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.

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

update script sources to be even with the example file ... #4121

Merged
merged 1 commit into from Jun 16, 2015
Merged

update script sources to be even with the example file ... #4121

merged 1 commit into from Jun 16, 2015

Conversation

cristovaov
Copy link
Contributor

... in reactjs/react-tutorial repo

cf. reactjs/react-tutorial#57

<script src="https://fb.me/react-{{site.react_version}}.js"></script>
<script src="https://fb.me/JSXTransformer-{{site.react_version}}.js"></script>
<script src="https://code.jquery.com/jquery-2.1.3.min.js"></script>
<script src="https://cdnjs.cloudflare.com/ajax/libs/react/0.13.0/react.js"></script>
Copy link
Member

Choose a reason for hiding this comment

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

We should leave this with {{site.react_version}} in there. That ensures we don't need to update this page for every release.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@zpao should be fixed with 4564e555b3b815f1ac8fcf9fd46eb61412b556aa ~

@zpao
Copy link
Member

zpao commented Jun 15, 2015

Great. 1 more request - could you flatten these to a single commit?

…ct-tutorial repo ... apply to japanese and korean translation (+2 squashed commit)

Squashed commit:

[4564e55] Ensures we don't need to update this page for every release.

[b0d60f9] update script sources to be even with the example file in reactjs/react-tutorial repo
@cristovaov
Copy link
Contributor Author

ping @zpao -- don't know if you had been notified that commits have been flatten as requested.

@zpao
Copy link
Member

zpao commented Jun 16, 2015

Ah, nope. Github doesn't send notifications for commit changes. Looks great, thanks!

zpao added a commit that referenced this pull request Jun 16, 2015
update script sources to be even with the example file ...
@zpao zpao merged commit f306ed6 into facebook:master Jun 16, 2015
@cristovaov
Copy link
Contributor Author

Cheers! You tested my git-via-sourcetree knowledge there, to squash commits to one ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants