These are chat archives for
Sign in to start talking
A bit torrent client
Jul 20 2015 14:25
Tests are failing on your PR. Please fix that and merge asap
Lets get this rolling again. I hope to Idle here a lot more often from now
Jul 20 2015 15:46
Jul 20 2015 15:47
I think I'll fix the test issues on
's PR and merge it, unless he comes back now
and does it himself
That is going to stagnate forever unless we do it now.
And sadly he came up with an insanely large PR
I'll carefully go through the code once anyway. Will file bugs for all the comments.
Jul 20 2015 15:55
Jul 20 2015 16:40
will do it tomo, meanwhile you go through PR
Jul 20 2015 16:43
Yes. I'll do that.
Jul 20 2015 16:51
, keep your history linear. Rebase on top of HEAD
Jul 20 2015 16:58
The topic branch diverged from master. the merge is not clean. So please rebase your changes on top.
Normally I dont say this, but in this particular case, it might be a good thing to squash the PR into one commit.
$ git reset --hard HEAD
$ git remote add origin
$ git fetch
$ git checkout dfordivam:server-design
$ git rebase origin/master
$ git push
A good commit history will help understand the code a lot in future. Please dont take it for granted
Jul 20 2015 17:05
Also I think you did your changes on top of some code by ramki. So this PR is a mix of everything.