-
Notifications
You must be signed in to change notification settings - Fork 572
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
Check on nedb usage and node >=23 #6538
base: master
Are you sure you want to change the base?
Conversation
I have spoken with @Ylianst he wants to keep nedb! |
If we implemented this PR it would break all existing setups so hense we looking for another module instead But also we should inform the user they can use |
Maybe already put a deprecation inform/warning in the current version, as a server message? I quickchecked the original nedb btw and at first glance it doesn't seem like a very big job to fix the deprecated util functions. |
If somebody already did it, better ofcourse! Which ones are you looking at? |
Added the warning. Just for completeness. Do you want to replace the module before the next release? |
@si458 Node 23 errors out if the current nedb module is used, so exitting gracefully if trying to use nedb seemed like the nicer option to offer with a clear message. But a replacement module would be better for sure. |
'fixes' #6503