tmattio / spin

OCaml project generator.
ISC License
297 stars 15 forks source link

Remove Lwt and Base #111

Closed tmattio closed 3 years ago

tmattio commented 3 years ago

This PR removes the dependency on Lwt and Base and updates to inquire.0.3.0 which now doesn't have any dependency.

The dependency on Lwt was unnecessary as there is absolutely no need for concurrency in Spin. Removing it makes the code much simpler and more readable. As part of this, I refactored the way we run subcommands to use spawn. The stdout and stderr are now redirected to the parent process, so we don't can display some progress to the user.

I also removed the dependency on Base. updated to latest version of inquire, which has no dependency. This is part of an effort to make Spin's dependency profile much leaner.