Christoph Kutzinski

@daniel-beck what do you mean by: "this code needs to make sure to work correctly with one thread"? Do you see any problems in that area? I think I accounted...

@daniel-beck I've implemented your suggested changes

I'm not sure, I yet understand what you mean by the common case, but I guess it's something like this: - you check out your main svn module to `.`...

@daniel-beck do you have any feedback?

Thanks for the PR. Sorry that this was left without attention for so long. Can you created a regression test case for the change? https://wiki.jenkins-ci.org/display/JENKINS/Unit+Test

This has been done, hasn't it? So can someone close the issue as it's confusing to still see it open?

So from looking at this very superficially: this PR tries to parse the revision.txt created by the svn-plugin? Yes, that's a no go IMO. So we should either provide an...

In fact - just ran into the same issue again - the logs on INFO level for successful and unsuccessful hook generation look exactly the same! Only the `GitHub webhooks...

Is one line of log more in the case of an error really spam? It would only log if the other line is logged as well, right?

If you have configured a user, but the user doesn't have the required permissions, I would consider it as an error. In contrast to when you don't have configured an...