raidenyn / yandexdisk.client

Yandex Disk API Client for .NET
MIT License
40 stars 14 forks source link

CLI for yandex.disk #5

Closed olegbevz closed 7 years ago

olegbevz commented 7 years ago

Hi,

I've added a new project YandexDisk.Client.CLI which implements two cli commands: yadisk.exe download a.xlsx C:\Users\User\Downloads -t yadisk.exe upload C:\a.xlsx Uploads -t

Project depends on CommandLineParser package to parse input arguments. I thought I could download yadisk.exe from nuget.org and added binary file to nuspec.

raidenyn commented 7 years ago

Oleg I have added unit tests for CLI but miss the one case for downloading folder with unzipping. I will add it during next few days or you can add it by yourself. Thanks.

olegbevz commented 7 years ago

Hi Yurij, unfortunately I have not so much time too, so I think it would be better if you finished with tests... I was thinking how I can import yasdisk.exe to my project: add nuget package or add *.exe manually to source control. Currently YandexDisk.Client.CLI depends on CommandLineParser nuget package, so if user wants to add YandexDisk.Client package he had to install CommandLineParser too. Perhaps we could embed CommandLineParser assembly into yadisk.exe (include it as embedded resourse). Or it is ok that YandexDisk.Client has this dependency?

raidenyn commented 7 years ago

Yes, you are right it is not so good. I think would be better to create a new nuget packages like YandexDisk.Client.CLI and to add all additional dependencies to it only. I think I'll do this in the near future.

AppVeyorBot commented 7 years ago

:x: Build yandexdisk.client 1.2.11-tavsqwre failed (commit https://github.com/raidenyn/yandexdisk.client/commit/f5f36961a2 by @raidenyn)

AppVeyorBot commented 7 years ago

:white_check_mark: Build yandexdisk.client 1.2.11-wcftwekb completed (commit https://github.com/raidenyn/yandexdisk.client/commit/3ae8b4b70c by @raidenyn)