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

Console removed - what to change? #219

Open
mauricioszabo opened this issue Sep 5, 2019 · 4 comments
Open

Console removed - what to change? #219

mauricioszabo opened this issue Sep 5, 2019 · 4 comments

Comments

@mauricioszabo
Copy link
Contributor

Today I found out that the old Console was removed on the new Ink version. How do I change the API? Is there any information about it? It broke completely my Chlorine plug-in, as it depends on a Console window currently opened...

The problem in this issue is that there's no documentation on how to migrate to InkTerminal and also no information on the CHANGELOG, so it's a huge breaking change with no visibility whatsoever...

@pfitzseb
Copy link
Member

pfitzseb commented Sep 5, 2019

The problem in this issue is that there's no documentation on how to migrate to InkTerminal and also no information on the CHANGELOG, so it's a huge breaking change with no visibility whatsoever...

Fair point. It's probably not really a good idea to treat ink as a library at this point, unfortunately.

There's no easy way to migrate, I think, but the easiest thing would be to directly start a REPL process and attach that to the InkTerminal. If you have only have stdin/stdout/stderr streams then things are probably more complicated, and I'm honestly not sure how that would end up working.

If you want to continue using the old console then I suppose we could re-add that, but I'd recommend just copying the relevant code (this, this, this, and this) over to your repo and drop the dependency on ink.

Let me know what you want to go with and I'd be happy to help.

@mauricioszabo
Copy link
Contributor Author

Well, I need the old functionality because I don't start any external process, so I need the stdout and stderr.

I was also using the console as a drop-in for ink results, so I need to render HTML (one thing that the new implementation don't support either). I'll probably just end up copying the code over my repo...

@pfitzseb
Copy link
Member

pfitzseb commented Sep 5, 2019

Alright. Sorry for the inconvenience then, but we can't really keep maintaining the old console and removed it to improve loading times.

@aviatesk
Copy link
Member

aviatesk commented Sep 8, 2019

@mauricioszabo
I missed to check this issue, and sorry for the inconvenience ... (I'm the one who removed the legacy console support)
But atst I think it might be better for your to just borrow REPL implementation into your repo as @pfitzseb pointed out, since the other Ink components (and the package interaction in general) are actually hurting your package loading/activation time (maybe A LOT).

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

No branches or pull requests

3 participants