Coffeescript integration #113
Conversation
| if language == 'coffeescript' | ||
| if language not in @loadedExtensions | ||
| @_execute '%load_ext coffee', => @loadedeExtensions.push language | ||
| "%%coffee bare no-prompt\n"+code |
There was a problem hiding this comment.
My main issue with this is that it flies in the face of the JavaScript kernel I'm currently using, ijavascript. It would be nice to unify the JS/Node kernels, otherwise we're going to have this special matrix of handlers for Hydrogen.
Then again, if there's a way we can tell which kernel is setup, we can defer to the right one.
|
@rgbkrk I just use nodejs kernel because there coffeescript is ready to integrate. This non-generic approach isn't good at all, there are many X->JS and I'll write something generic when have a time for that. |
|
Thinking on this, I might make a PR to the notable mind js kernel to have coffeescript, babel, etc. be kernelspecs you can install instead of using magics. |
|
would be great |
|
My guiding principle is to not include any language-specific code in Hydrogen. Kyle's suggestion sounds like the best way to deal with this — that's what kernelspecs are for, after all! I also want to have a way to switch which kernel you're using with a file, but I'm swamped right now, so I'm not sure when that's going to happen. @rgbkrk having a single Babel kernel would be awesome. |
|
@rgbkrk Notify me, please, when you finish with kernels. I'll implement hydrogen part |
|
Finally we can use this: https://github.com/n-riesco/jp-coffeescript |
works with https://github.com/terraqout/jupyter-nodejs until notablemind/jupyter-nodejs#16 isn't merged
Code review and any help appreciated