-
Notifications
You must be signed in to change notification settings - Fork 82
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
Enable FTP with SSL support #62
Conversation
This option is required in order to have FTP with SSL support.
@hajekj Hi, thank you for a pull request. It's very helpful for me that you are engaged in this project too. What I'd like to ask you is to apply to your fix the same indentation as in original file. Right now I cant really check what was changed because it looks like the whole file was rewritten. |
Generally, I think that the formatting is screwed up in the file, all that was changed is following:
|
Hi @hajekj |
@Nighthawk22 That is a great idea, I will see about adding that into this PR. |
@hajekj @lukasz-wronski Can you please finish this pull request? I see it's just a small change. It would be very helpful! Thanks a lot. |
Sorry, don't have permissions on the repo. |
@lukasz-wronski Can't you give permissions to someone else as well? The latest update of this repo is from June :( It's a pitty. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This should be easy to merge.
We could fork the repo and make our own official release from this.. As a plan B |
Lukasz is looking for collaborators #96. I think you can just ask him to get access |
pt., 21.10.2016, 23:46 użytkownik Jan Hajek [email protected]
|
This option is required in order to have FTP with SSL support.
Fixes #30