search

koalazak / dorita980

Unofficial iRobot Roomba and Braava (i7/i7+, 980, 960, 900, e5, 690, 675, m6, etc) node.js library (SDK) to control your robot
MIT License
931 stars 147 forks source link

Install instructions unclear. #183

Closed CrysKilljoy closed 7 months ago

CrysKilljoy commented 7 months ago

was looking for node.js on every homeassistant library. until i found that it's a windwos tool.

aft pasting the code into node on windows i only get Uncaught SyntaxError: Unexpected identifier 'npm'

CrysKilljoy commented 7 months ago

after trying around, you should not opne node.js directly but the command prompt. and you shall not use the "$" in the presented code so just paste "npm install dorita980 --save"

now i get 

C:\Users\chrja>npm install dorita980 --save
npm WARN deprecated har-validator@5.1.5: this library is no longer supported
npm WARN deprecated request-promise@4.2.6: request-promise has been deprecated because it extends the now deprecated request package, see https://github.com/request/request/issues/3142
npm WARN deprecated uuid@3.4.0: Please upgrade  to version 7 or higher.  Older versions may use Math.random() in certain circumstances, which is known to be problematic.  See https://v8.dev/blog/math-random for details.
npm WARN deprecated request@2.88.2: request has been deprecated, see https://github.com/request/request/issues/3142

added 126 packages in 9s

4 packages are looking for funding
  run `npm fund` for details
npm notice
npm notice New minor version of npm available! 10.1.0 -> 10.2.4
npm notice Changelog: https://github.com/npm/cli/releases/tag/v10.2.4
npm notice Run npm install -g npm@10.2.4 to update!
npm notice
koalazak commented 7 months ago

hi @CrysKilljoy, this library is for node.js developers. For them the documentation is pretty clear. regards,