[CLOSED] Update to syrup 0.1.3

[sbhaktha]

Issue by codeviking Fri Apr 3 19:29:25 2015 Originally opened as https://github.com/allenai/okcorpus/pull/75

---

@afader @chrisc36 @dirkgr

This applies the latest syrup version, which has some logging and speed improvements.

A few notes:

• I added a .jshintrc file such that jshint "plays" well with jsx -- but I kept the javascript linter disabled. If you want to enable it simply edit the gulpfile and remove disableJsHint: true. I tried doing this but there's a lot of lint errors right now -- at one point in the near term we should resolve these and renable the linter (as it helps catch errors prior to even running the code in the browser).
• You can now run gulp without a global executable, the two commands are npm run build and npm run watch. These are equivalent to gulp build and gulp watch, respectively.
• After this is merged, anyone with an existing installation will unfortunately have to re-run npm install in webapp. Things won't work if you don't (as this syrup version is incompatible with 0.0.*).
• There's bound to be a few small issues, let me know if you counter them and I'll get them patched right away. Thanks for being my test set :)

---

codeviking included the following code: https://github.com/allenai/okcorpus/pull/75/commits

[sbhaktha]

Comment by codeviking Fri Apr 3 19:29:41 2015

---

@afader I'm assigning this to you for a quick review.

[sbhaktha]

Comment by codeviking Fri Apr 3 19:32:34 2015

---

Oh, also I failed to note that I switched the build from reactify to babel. Babel enables ES6 syntax if you guys choose to use it. Things like Map, Set, etc...

https://babeljs.io/

[sbhaktha]

Comment by dirkgr Fri Apr 3 20:17:43 2015

---

Won’t this make things less debuggable, since we’re debugging the compiled code as it is?

On April 3, 2015 at 12:32:36, Sam Skjonsberg (notifications@github.com) wrote:

Oh, also I failed to note that I switched the build from reactify to babel. Babel enables ES6 syntax if you guys choose to use it. Things like Map, Set, etc...

https://babeljs.io/

— Reply to this email directly or view it on GitHub.

[sbhaktha]

Comment by codeviking Fri Apr 3 20:18:08 2015

---

@dirkgr Nope! Source maps ftw!

(This syrup release provides source map support and support for the debugger keyword).

Also, if you guys feel like the build is slow you can toggle compressJs: false in the gulpfile to speed tings up. The slowest part of the build currently is the compression step.

[sbhaktha]

Comment by codeviking Tue Apr 7 14:40:17 2015

---

Ping! Do we want this merged in?

[sbhaktha]

Comment by dirkgr Tue Apr 7 16:44:57 2015

---

:+1:

[sbhaktha]

Comment by afader Tue Apr 7 19:06:32 2015

---

Looks good, thanks for the help, @codeviking.