search

daveallie / entangler

Two way file syncer using platform native notify
MIT License
8 stars 6 forks source link

Remove rvm requirement #10

Closed codersasha closed 7 months ago

codersasha commented 7 months ago

Running entangler runs source ~/.rvm/environments/default, which fails if rvm is not installed.

Remove this requirement, allowing entangler to run without rvm.

daveallie commented 7 months ago

Hey @codersasha, I don't mind adding this functionality, but would prefer it in a slightly different format.

I've made some changes over here: https://github.com/daveallie/entangler/pull/11, would you mind confirming that they solve your usecase? If so I'd prefer to merge that one over this PR, as it covers a few things:

  1. Avoids trying to run source when the file doesn't exist
  2. Allows users to opt out of the check all together if they want
daveallie commented 7 months ago

Closed by #11