I miss the "Ship It" button...

+1, This seems to be the motivation because my transfers fail if the connection is not 100% perfect.

In D13580#300886, @pprkut wrote:

I rebased this against master and found it has the same problem as D13547, i.e. it introduces 10 first/follow conflicts :/
Not sure it's worth trying to solve it here either.

Hello.
Sorry for the late reply, I haven't noticed when this review was created

This adds support for generators and generator delegation

In D11785#245911, @dhaumann wrote:

Indeed, maybe this class should be part of the KConfigWidgets framework? See https://api.kde.org/frameworks/kconfigwidgets/html/annotated.html for current list of classes - it already contains the color scheme stuff and is a tier3 framework (so many dependencies allowed).

@zhigalin Any comments from your side?

Don't know if this will help but I use two combined plasmoids to achieve this nice effect:

@santilin
I would like to merge it, including schema defaulting to http:// as I have proposed,
any objections?

@santilin It seems you did a duplicate here, please close D8555

@bcooksley Any news?
I didn't noticed that because I was on the KDevelop mailing list and I was receiving replies from it but it seems that I have never received any mail from phabricator...

@apol You're right, wasn't thinking that way...

In D8088#152182, @bcooksley wrote:

I've taken an initial look at that and can confirm the issue, although not why it is occurring and have reached out to the Phabricator developers.

@bcooksley when I visit email settings page https://phabricator.kde.org/settings/user/zhigalin/page/email/ I see only this error:

In D8088#152167, @bcooksley wrote:

The creator of a review is an implicit subscriber from my understanding of how Phabricator works here.

@bcooksley Why I am not automatically subscribed to a review I have created?

In D8088#151644, @apol wrote:

I think it's good to show the dialog if there's just the one device. This way you see what's happening, at least that was my thinking.

Remove scripts on windows

In D7863#147030, @kfunk wrote:

b) Limits this feature to hastebin.com

Why hastebin?
Two reasons:

1. Don't require a client, a single shell command is enough
2. It's open source! So It isn't really limited to hastebin.com, anyone could set up his own hastebin instance...

Ping

Rest LGTM, do you have commit rights?

Sorry, had no time lately

function foo($a, $b = false, $c) {}
Should work?
Now it says that $c should have a default...

Hello
I should note that

Update

Way better then the previous hack.
@kfunk?

Rest LGTM

New version

Perfect, thanks

Cool! Seems we were working on the same issue
As mentioned by @apol and @kfunk a test is needed.

In D6807#127164, @apol wrote:

Maybe you could add a unit test?

Good one, pushing it right now...

Nice one, thanks

Looks good, thanks for your work!