On the runtimes reviews

Milian Wolff mail at milianw.de
Wed May 24 13:04:34 UTC 2017


On Monday, May 22, 2017 9:29:10 AM CEST Sven Brauch wrote:
> Hey,
> 
> I only skimmed the patch but read the discussion, and it seems sensible
> to me. I'd say feel free to put it into master and let's see if anything
> breaks ... ;)
> 
> Best,
> Sven
> 
> On 22/05/17 00:22, Aleix Pol wrote:
> > Hey guys,
> > I understand eveyone's busy and it's not super fun to go through a
> > massive patch such as these (I know Milian did, thanks a ton for it!).
> > Since I think it's something good to start integrating anyway, I think
> > it could be interesting to just merge it to master so you guys can
> > test it. In general nothing should change if you don't tinkle with it
> > and I'll work on any issues that we might find before the next
> > release.
> > 
> > Thoughts?
> > Aleix

Yep, good to see that you integrated it. We can always clean up afterwards, if 
needed. Better than bitrotting in review hell.

Cheers

-- 
Milian Wolff
mail at milianw.de
http://milianw.de


More information about the KDevelop-devel mailing list