Damm Kewl (9b020a50) at 07 Feb 21:51
Removed erroneous recursive flag on cp command used for copying a f...
... and 7 more commits
Damm Kewl (7fc84e3d) at 01 Feb 22:35
dev commit
Update nodejs to v20.x and deps.
BTCxZelko (1fba212a) at 10 Jan 02:54
Merge branch 'feature/nodejs_version_bump' into 'develop'
... and 3 more commits
These are the commands provided by nodejs.
Update nodejs to v20.x and deps.
This comment is non-blocking, changes not mandatory:
Do you need the value of this echo in the terminal output? if not, you can use ">" instead of "| tee"
tee's generally used to get sudo to work in a scenario where you need ">", or if you need to simultaneously both pipe a value to some process as well as put it in a file.
Update nodejs to v20.x and deps.
BTCxZelko (819697d9) at 07 Jan 21:24
fix recommendations
In that case I'd like the use-case for this new dependency first before approving its addition.
Can be combined
npm i -g pnpm@7 pm2
Not a requirement for Ronin UI afaik
it can be added to the migration proposed in RoninDojo!281
Is this a requirement for RoninUI? Does RoninUI intend to add showing nvme temperatures for example? If so, we'll need to also create a migration in RoninDojo for existing user to obtain this package.
this change to does not belong in this MR
Update nodejs to v20.x and deps.
BTCxZelko (02473a6c) at 28 Dec 23:31
clean up and lock pnpm to v7
BTCxZelko (ce6ce27f) at 28 Dec 23:27