-
Notifications
You must be signed in to change notification settings - Fork 170
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[FIXED JENKINS-16207] TFS 2012+ Links Fixed #26
Conversation
Conflicts: .gitignore
…r password" This reverts commit d4d7dff.
Not sure when this changed as it was not a problem a few months ago.
Thank you for a pull request! Please check this document for how the Jenkins project handles pull requests |
Sorry for the very long delay; when I first got commit access to this project, I had been only tackling my immediate needs at $DAYJOB. I am planning to review these changes for release 3.3.0. |
I would love to get this one into a release soon! +1 |
@olivierdagenais I was just checking the repo browser URL persistence on a real server and found that we need this merged for links to changesets and so on to actually work. Are you mostly okay with how this PR is implemented? If yes, I will clean it up into a separate branch and then test and merge it. |
Also might default the new parameter to true so that it will work out of the box for new TFS server installations.. |
Given that we do not even support older versions out of the box I think this might be overkill. Instead of a config flag and so on we might want to just change the URL to use the new URLs. Can you confirm that these style URLs are used in all current versions? I will prepare a PR for this approach next. |
Replaced by #71 |
Fixed the merge issues with the last request. Option exists under the Adanced section to use the REST style urls created with TFS 2012+