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

Support For React18, Updated Dependencies #24

Open
wants to merge 12 commits into
base: master
Choose a base branch
from

Conversation

jaxtonw
Copy link

@jaxtonw jaxtonw commented Apr 14, 2023

Updated the project and all it's dependencies to build again and support React 18. Previous packages were deprecated and the project in it's current state did not build. This has since been fixed, and with updated dependencies, it should hopefully last for a while.

The version has been updated to 1.1.0. I debated versioning to 2.0.0, but this shouldn't cause any major breaking changes for React 16/17 to my knowledge.

I apologize if any of the configuration changes when updating dependencies conflict with the initial plans of the project. I struggled a bit in trying to keep things as similar as similar as possible while not being able to build the old version.

Any critiques or suggestions are appreciated!

@jaxtonw
Copy link
Author

jaxtonw commented Apr 24, 2023

The previous commits from when this PR was opened did not actually work with TypeScript. This issue stems from webpack. Just compiling the project with tsc caused everything to work properly, and also resulted in a smaller distribution. The previous two commits accomplish that.

@jaxtonw
Copy link
Author

jaxtonw commented May 7, 2023

@robert-harbison has there been any updates on this pull request?

@jaxtonw
Copy link
Author

jaxtonw commented May 30, 2023

@robert-harbison have you had a chance to look at this yet? People are starting to suggest name-squatting packages in order to fulfill what this PR attempts to accomplish.

#22 (comment)

@JensvandeWiel
Copy link

@robert-harbison please take a look at this and update it.

@kylebonnici
Copy link

kylebonnici commented Oct 3, 2023

@robert-harbison any plans to merge this ?

I have just tried this out by building local and it works :) nice work @jaxtonw.

@JensvandeWiel
Copy link

Someone should make a good fork and maintain it :/

@PabloLION
Copy link

Hi @jaxtonw, I made a new repo (with react18 support) for the same purpose on https://github.com/PabloLION/xterm-react and I'm trying to improve it. I like the tests you've written. Would you mind to make a similar PR in my repo, or if I copy them from this PR directly?
Thanks for the hard work to the open source community!

P.S. I have an end-to-end test with vite, and I'm planning to host it with GitHub Pages as a live example. You might want to consider how to integrate the test with that as well

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

Successfully merging this pull request may close these issues.

4 participants