BTW a tip for minimizing diff lines in a PR: avoid changing indentation of long blocks of existing code if it can be helped:

```
    boolean upgradeIfNecessary(AbstractProject<?,?> job) throws IOException {
        if (isUpgradeNeeded()) {
            int i = projectName.lastIndexOf('/');
            …
            return true;
        } else {
            return false;
        }
    }
```

This practice makes the PR easier to review, makes history easier to follow (especially from `git blame`), and makes merge conflicts less likely.