Skip to content

show upload progress for paket push - #695

Merged
forki merged 7 commits into
masterfrom
unknown repository
Mar 12, 2015
Merged

show upload progress for paket push#695
forki merged 7 commits into
masterfrom
unknown repository

Conversation

@ghost

@ghost ghost commented Mar 11, 2015

Copy link
Copy Markdown

this should work. Not so sure about the dependency onFSharp.Control.Reactive but a sampling operator is nice for this

Comment thread src/Paket.Core/Paket.Core.fsproj Outdated

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

woups? what happened here?

@ghost

ghost commented Mar 11, 2015

Copy link
Copy Markdown
Author

Yes that is at least what it should do conceptually, since not every buffer update should be reported. I can try to achieve the same without the dependency, it will be just a bit hairy.

@forki

forki commented Mar 11, 2015

Copy link
Copy Markdown
Member

just dump the hairy parts into Utils.fs like we all do ;-)

@ghost

ghost commented Mar 11, 2015

Copy link
Copy Markdown
Author

ok :-)

@forki

forki commented Mar 12, 2015

Copy link
Copy Markdown
Member

is this done?

@ghost

ghost commented Mar 12, 2015

Copy link
Copy Markdown
Author

it's done now, I just added upload speed and completion percentage

@forki

forki commented Mar 12, 2015

Copy link
Copy Markdown
Member

wow cool. could you please remove the ol UploadFileAsMultipart?

@forki
forki merged commit a2d2735 into fsprojects:master Mar 12, 2015
@forki

forki commented Mar 12, 2015

Copy link
Copy Markdown
Member

mhm. I just merged it and tried it. It didn't show any progress at all.

image

@forki

forki commented Mar 13, 2015

Copy link
Copy Markdown
Member

@kfrie any ideas?

@ghost

ghost commented Mar 13, 2015

Copy link
Copy Markdown
Author

@forki the sample operator was blocked by push |> Async.RunSynchronously. I fixed this, can you reopen please?

@forki

forki commented Mar 13, 2015

Copy link
Copy Markdown
Member

I can't reopen. But you can send a new pull request. And many thanks

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.

1 participant