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

add: support for node v14, v16 #33

Closed
wants to merge 2 commits into from

Conversation

Arch0125
Copy link

No description provided.

@dagnelies
Copy link
Collaborator

Hi, I struggled quite a lot to make the same build work for browsers and node and cloudflare workers, etc. I'm also pretty sure changing the --platform=neutral to --platform=node will bust the bundle for several of such platforms. 💣

Moreover, node 14 and 16 are deprecated, they reached end-of-life and don't even receive security fixes anymore. Therefore, I'm gonna ignore them.

Node 18 support would have been nice, but on the other hand, it would imply that all other platforms pull one more dependency (is it even a cross platform dependency working well in browsers, cloudflare workers, deno, etc.?).

Because of these reasons, I'll close the PR. Thanks for the effort though. If you want, I can link your fork in the README for those looking for older Node versions compatibility.

@dagnelies dagnelies closed this Sep 21, 2023
@Arch0125
Copy link
Author

Thanks for going through the PR
Yeah definitely changing --platform=neutral to --platform=node might break build for several other platforms as well, however I was just targeting the node platform with this PR.

Regarding support, yeah it does support Node v18 as well, but still as you mentioned it would still require crypto as a outside dependency and as far other platforms go i'm looking into them

It'll be great if you drop the link in the Readme as I've seen many projects still running on Node v16 if not v14

@dagnelies
Copy link
Collaborator

Ok, let's do that 👍

@dagnelies
Copy link
Collaborator

Added following comment to README:

(*) For older Node versions, take a look at Arch0125's fork. (The reason of the Node 19+ compatibility is basically WebCrypto being globally available, making it possible to use the same build for all targets: browser, node, clouflare workers...)

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.

None yet

2 participants