search

piotrmurach / tty

Toolkit for developing sleek command line apps.
https://ttytoolkit.org
MIT License
2.5k stars 78 forks source link

#57 Escape app path #67

Closed d3mash closed 4 years ago

d3mash commented 4 years ago

Describe the change

Escaping app path via Shellowords#shellescape

Why are we doing this?

Because of Issue #57

Benefits

This will allow to create new teletype apps in directories containing whitespaces

Drawbacks

Possible drawbacks applying this change.

Requirements

Put an X between brackets on each line if you have done the item: [x] Tests written & passing locally? [x] Code style checked? [x] Rebased with master branch? [x] Documentaion updated? [no need]

d3mash commented 4 years ago

Done 👍 I should have noticed that it is project-style not to use RSpec hooks 😄

piotrmurach commented 4 years ago

Thanks! ❤️