Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implemented ability to open a folder #44

Closed

Conversation

KES777
Copy link
Contributor

@KES777 KES777 commented Sep 21, 2015

@aurora
Copy link
Owner

aurora commented Oct 1, 2015

Thanks for your patch! I've always rejected this feature in the past because i think you can easily do this already in combination with other commands like find. Anyway ... i leave this open and play around with it ... maybe i'll change my mind! 😉

@KES777
Copy link
Contributor Author

KES777 commented Oct 2, 2015

oh, you can replace my code by line using find =). I use linux under 10 years and I do not know how to use find. I need to google, read some article ... ((( it is too hard vs: rsub dir/

@ghost
Copy link

ghost commented Jun 1, 2016

This is how to open many files in a folder in many tabs in SublimeText. But how to open a folder, like subl . command?

@KES777
Copy link
Contributor Author

KES777 commented Jun 2, 2016

@ginolon : yes, that is interesting feature. but not supported yet. There is bounty
Join ;-)

@ickc
Copy link

ickc commented Nov 5, 2017

This line is not safe: for dir in `ls $filepath` ; do. e.g. use find ... -exec or find ... -print0 | xargs -0 ... instead.

@aurora
Copy link
Owner

aurora commented Jan 6, 2018

I am closing this pull request, as there's a merge-conflict currently anyway. Also i think the functionality should be implemented using features of the current rmate version (using the open_file function rather than spawning additional rmate processes). I have some ideas regarding this and am working on an own patch to support this now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants