-
Notifications
You must be signed in to change notification settings - Fork 22
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 to React 0.11 #3
Conversation
Did you run the tests and verify a source map shows up? I don't know the new options, but this shim probably will need some changes if it doesn't. https://github.com/ddispaltro/sbt-reactjs/blob/master/src/main/resources/jsx.js I copied it as a template from the coffeescript compiler. Thanks for the contributions! |
Actually I couldn't do it yet because I'm not familar with sbt scripted |
ill give it a go, standby |
Just fixed a JS param name and checked it working. If you want some tests I'll do it in a few days. |
Man something happen with logger inclusion in latest sbt plugin |
Actually I'm not sure the web browsers understand inline source map, but the source map can be attached by the latest commit anyway. I'll revert play version and add some tests to this PR when I have some spare time. BTW, do you have any plan for automated test via Travis CI? |
No plan, haven't thought about the travis stuff really at all. Is it possible to test sbt plugins with travis? Btw to run the tests you just do
|
Sure, you can override test command using |
Added |
Im going to merge and test travis |
Update to React 0.11 with source maps.
Travis is working, I had to edit the jvmopts to give it more memory and move over to |
Great, glad to contribute :) |
Pushed 0.3.0 to bintray. On Tue, Jul 22, 2014 at 5:22 PM, Jisoo Park [email protected]
Dan Di Spaltro |
sourceMapInline
option