search

abicky / nodejs-repl.el

Run Node.js REPL and communicate with the process
190 stars 38 forks source link

Trying ts-node REPL #10

Open arichiardi opened 7 years ago

arichiardi commented 7 years ago

Hello! Thanks for this nice package first of all :smile:

I know this is not really supported but it would be great to have typescript Repl. I am still considering if it is worth modifying this or having a completely new inf-typescript for instance. I think the sexp detection part can be difficult to achieve for typescript, and should be probably delegated to typescript.el.

Anyways, I am trying it out and I get:

⨯ Unable to compile TypeScript
[eval].ts (1,16): Property 'columns' does not exist on type 'WritableStream'. (2339)

Process nodejs exited abnormally with code 1

Checking the code a bit, it seems that the new process.stdout (I am using node 7.9.0) does not have columns anymore. In general probably that block causes the problem and it is not required anymore form new repls. I might be wrong of course.

Would you be interestend in a defcustom that avoids passing the -e to the process?

In general I'd be interested in your opinion on an alternative Emacs mode/package.

arichiardi commented 7 years ago

Uhm, this is actually related: https://github.com/DefinitelyTyped/DefinitelyTyped/issues/13508