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

Update package.json #508

Closed
wants to merge 1 commit into from
Closed

Update package.json #508

wants to merge 1 commit into from

Conversation

AutoStake-com
Copy link
Contributor

@AutoStake-com AutoStake-com commented May 31, 2022

NODE_OPTIONS=--openssl-legacy-provider
produces error:
node: --openssl-legacy-provider is not allowed in NODE_OPTIONS
at least on node 16.X (Current LTS)

`NODE_OPTIONS=--openssl-legacy-provider` produces error: "node: --openssl-legacy-provider is not allowed in NODE_OPTIONS" at least on node 16.X (Current LTS)
@tombeynon
Copy link
Contributor

Hmm that's annoying. Master uses node 17 which doesn't work without this config 🤔 The new EthWallet package requires this config unfortunately.

It's a shame to downgrade, are you able to upgrade to node 17 comfortably?

@AutoStake-com
Copy link
Contributor Author

Yes, that's fine no problem, perhaps just a notice in the README for others, I will submit a PR in the morning, getting late now
thanks again

@tombeynon
Copy link
Contributor

Awesome thanks dude, I didn't think about people running without Docker. We'll keep this PR open for now for visibility.

@AutoStake-com
Copy link
Contributor Author

We do run within our own custom Docker, which makes upgrading to v17 easy, it's just our default to select an LTS normally without any other reason not to

@tombeynon
Copy link
Contributor

Ahah makes sense. I did also forget about the non-docker users though 😂

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