feathersjs / feathers

The API and real-time application framework
https://feathersjs.com
MIT License
15.03k stars 748 forks source link

Update docs for those without feathers cli in dev/globals #3121

Closed FossPrime closed 1 year ago

FossPrime commented 1 year ago

Summary

This could work 100% of the time... but right now it only works IF you have feathersjs/cli installed globally or in the devDependencies. If you don't you'll have to know it's in @feathersjs/cli somehow. That's specially not obvious as the Getting Started guide uses create-feathers not the CLI explicitly.

Daffl could replace https://www.npmjs.com/package/feathers with the CLI to make it work... Perhaps it's worth cross-publishing.

daffl commented 1 year ago

If you follow the guide as intended that command should always work since a generated v5 app will always install the CLI as a devDependency.

daffl commented 1 year ago

I'd still be curious if you ran into a problem with it? I know a few people had issues with the old CLI still being installed globally.

FossPrime commented 1 year ago

I was guiding a newbie on Feathers CLI and he managed to get this while generating a service... not sure if it was because originally the app was created with custom service, or because of some SIGKILL interruption. I don't use the CLI much as 99% of the time, I'm making a prototype and don't want to deal with binary DB's, but do want a real adapter like feathers-memory or what feathers-NeDB used to be. If I see the issue again I'll report.

Screenshot 2023-03-21 14 51 27