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

Use babel preset in cli #302

Merged
merged 1 commit into from
Jul 15, 2016
Merged

Conversation

doug-wade
Copy link
Collaborator

Transpile the cli (but not its clients) with babel-preset-react-server

@doug-wade
Copy link
Collaborator Author

This is blocking on 0.4.0 to publish

@gigabo
Copy link
Contributor

gigabo commented Jun 29, 2016

@doug-wade - Blocking on 0.4.0 of what?

@doug-wade
Copy link
Collaborator Author

@gigabo React Server!

@gigabo
Copy link
Contributor

gigabo commented Jun 29, 2016

It requires breaking changes?

@doug-wade
Copy link
Collaborator Author

doug-wade commented Jun 29, 2016

Whichever version we publish next. Semver versions before 0 are kinda squishy -- adding two new packages to the ecosystem seemed like it might merit more than patch version change.

@doug-wade doug-wade force-pushed the use-babel-preset-in-cli branch 2 times, most recently from 15b8bc7 to 454f32f Compare July 1, 2016 01:04
@@ -38,7 +38,7 @@
"jasmine": "2.3.1",
"jsdom": "5.6.1",
"jsx-loader": "0.12.2",
"lerna": "2.0.0-beta.23",
"lerna": "git://github.com/lerna/lerna.git#master",
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

build is passing locally with my lerna built off master (which runs prepublish as part of bootstrap), but not on ci so I tried pulling from master. However, lerna's master branch isn't a functioning node module, so this isn't working either. Next, I'll try a lerna run prepublish on ci, but if that doesn't work, we may just need to wait for lerna-2.0.0-beta.24

@doug-wade doug-wade force-pushed the use-babel-preset-in-cli branch from 454f32f to bc8ee85 Compare July 15, 2016 20:02
@doug-wade doug-wade force-pushed the use-babel-preset-in-cli branch from bc8ee85 to efbc7a3 Compare July 15, 2016 21:39
@doug-wade doug-wade force-pushed the use-babel-preset-in-cli branch from efbc7a3 to 12133f1 Compare July 15, 2016 22:21
@doug-wade doug-wade merged commit f230bec into redfin:master Jul 15, 2016
davidalber pushed a commit to davidalber/react-server that referenced this pull request Jul 24, 2016
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.

2 participants