-
Notifications
You must be signed in to change notification settings - Fork 626
Updated the JavaScript files code from CommonJS to EC Modules #458
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
base: master
Are you sure you want to change the base?
Conversation
|
Please continúe working on that, this is the kind of updates i would like to keep seeing on NodeOS :-) Have you been able to update the Node.ja version too? |
right now i'm working on a personal project but i've been able to modify some of the code, i'll compile everything and run it using a virtual machine to see if everything works properly |
|
everything is up to date and should be working properly, once i'll have tested everything and made sure than it works i'll confirm than it works here |
|
i modified all i've done to make sure than there's absolutely no bug when it's running and made it so that rather than using multiple variables to call all the different functions of the dependancies it's instead included in an array called dependancies i also modified all the .sh files so that the license is up to date |
|
now everything is good, i just need to compile everything and test it on a VM and i'll see if it works properly or not |
|
well i have to update the dependancies because some of them are totally out of date and it's causing npm to return errors, it's going to be fun |
Updated all the :
const var_name = require('library_name');
const ineer_namespace_name = require('library_name').inneer_namespace;
to :
const var_name = await import("library_name");
const {ineer_namespace_name} = await import("library_name");
If NodeJS itself has the required libraries then i used node:library_name to not have to download the libraries through npm and save bandwith and time