cole-h / agenix-cli

Companion tool to https://github.com/ryantm/agenix
Apache License 2.0
31 stars 3 forks source link

feat: split EDITOR on ' ' to find binary name #3

Closed ryantm closed 3 years ago

ryantm commented 3 years ago

fixes #2 by just implementing the split method.

Caveat: I know very little rust, so this might be dumb.

cole-h commented 3 years ago

Thanks! I did something similar in 90dc2ea5510842d70dde74027da1d81942e9e00c.

One problem with this PR was that .remove(0) will panic if run with EDITOR= (can't remove the 0th element of a 0 element list), so I did something slightly more robust (IMHO).