-
Notifications
You must be signed in to change notification settings - Fork 3
added RMD command #42
base: master
Are you sure you want to change the base?
Conversation
hey, nice to meet you! sorry for the late reply, I haven't had the chance to take a look at it yet! I'll take a in-depth look this week, but it looks good! Thanks a lot for your help, I'm super excited! |
Thank you for your reply!! Thank you :-)) |
@koenw are you still going to process this PR? Looks like a useful feature to me ;-) |
Hi @hannesdejager. I rebased to master. Thank you ;-) |
Thanks @omikuji, the travis runs are failing, seems like you need to run the formatter and check in again :-) |
Hi @hannesdejager. I'm sorry :_( challenge again! |
@omikuji since Koen is not responding and no one else have rights to merge your contributions you may want to consider to submit your contributions to our fork here: https://github.com/bolcom/firetrap. This project is active, we have 4 people working on it on a weekly basis. You can make pull requests to our develop Branch. Also we're busy with an FTP server that is making use of our forked library too, find it here: https://github.com/bolcom/unFTP Regards |
@hannesdejager thank you! I am so very happy! I'll be right over! |
Nice to meet you!
I added an RMD command to delete the directory.
Thank you.