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

Add babel plugin for module tagging #280

Closed
wants to merge 12 commits into from

Conversation

doug-wade
Copy link
Collaborator

@doug-wade doug-wade commented Jun 9, 2016

Add babel-plugin-react-server to the react-server-cli webpack loader and
babel-preset-react-server. Factor out react-server-module-tagger from
react-server-gulp-module-tagger to share module tagging format between
gulp and babel, and add tests to ensure prevent regressions. As discussed in #276; fixes #12 and #247

@CLAassistant
Copy link

CLAassistant commented Jun 10, 2016

CLA assistant check
All committers have signed the CLA.

gigabo and others added 10 commits June 13, 2016 10:26
* Upgrade lerna-2.0.0-beta.18

This picks up some patches that will help us out:

- lerna/lerna#181
- lerna/lerna#196
- lerna/lerna#198
- lerna/lerna#199

* Use `lerna clean` for `npm run nuke`
Previously ClientController's nodeArrival function just took in the node index,
but this made it such that each arriving node required its own script tag and
LABjs callback; now nodeArrival can take in a range of indices to minimize
overhead.
Porting the changes from the main README to the react-server README
Reduce nodeArrival calls by taking in range of node indices
aickin and others added 2 commits June 15, 2016 17:19
Adds babel-plugin-react-server to the react-server-cli webpack loader and
babel-preset-react-server.  Factor out react-server-module-tagger from
react-server-gulp-module-tagger to share module tagging format between
gulp and babel, and add tests to ensure prevent regressions.
@doug-wade
Copy link
Collaborator Author

This has gotten unwieldy, so I'm breaking it into multiple prs (see #300, #294, and #297, with more to follow)

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

Successfully merging this pull request may close these issues.

5 participants