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

NodeJS 22 install error #539

Closed
ghost opened this issue Nov 5, 2024 · 9 comments
Closed

NodeJS 22 install error #539

ghost opened this issue Nov 5, 2024 · 9 comments
Labels
bug Indicates an unexpected problem or unintended behavior.

Comments

@ghost
Copy link

ghost commented Nov 5, 2024

The .nvmrc file specifies that the version to be used is lts/*. , but since October 29, 2024, version 22 has been upgraded to LTS.

This leads to a problem when executing the npm install command, which is as follows:

import { LogHelper } from '@/helpers/log-helper
         ^^^^^^^^^
SyntaxError: The requested module '@/helpers/log-helper' does not provide an export named 'LogHelper'.

With some quick research I could see that some other projects had similar issues, but I couldn't identify a solution :/.

I think a change is needed in the future so that new contributors are not discouraged.

In any case, I'd like to thank you Louis for this project, which I discovered a few years ago. I'll try to help out in my spare time, as the PDA is the last barrier for those who want to do without proprietary solutions ^^.

@ghost ghost added the bug Indicates an unexpected problem or unintended behavior. label Nov 5, 2024
@gagipro
Copy link

gagipro commented Nov 9, 2024

yes encountered same issue for the moment. need to wait a bit I suppose.

@ghost
Copy link
Author

ghost commented Nov 9, 2024

yes encountered same issue for the moment. need to wait a bit I suppose.

You can downgrade to v20 of NodeJS. It works with this version.

@gagipro
Copy link

gagipro commented Nov 9, 2024

Ok will try

@gagipro
Copy link

gagipro commented Nov 9, 2024

yes encountered same issue for the moment. need to wait a bit I suppose.

You can downgrade to v20 of NodeJS. It works with this version.

I confirm it works with node v20.
Thanks a lot.

@ghost
Copy link
Author

ghost commented Nov 9, 2024

You're welcome ^^.

@louistiti
Copy link
Member

@louistiti
Copy link
Member

Hi @Arthur-GYT @gagipro, thanks for the report.

I'm coming back to work on Leon, I pushed a fix on the develop branch to support Node.js 22. Can you please help try again?

@gagipro
Copy link

gagipro commented Nov 16, 2024

Hi @Arthur-GYT @gagipro, thanks for the report.

I'm coming back to work on Leon, I pushed a fix on the develop branch to support Node.js 22. Can you please help try again?

Hello Louis, yes I will try as soon as I have finished my other preparation on other project.

Thanks a lot

@ghost
Copy link
Author

ghost commented Nov 16, 2024

It's good for me @louistiti. Thanks for your fix

@ghost ghost closed this as completed Nov 16, 2024
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Indicates an unexpected problem or unintended behavior.
Projects
None yet
Development

No branches or pull requests

2 participants