Skip to content

Update jenkinsci repository with PRs merged into original author's repo #23

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

Closed
wants to merge 5 commits into from

Conversation

jimklimov
Copy link

@nemccarthy "upstream" repository has integrated a couple of PRs from here and there, a year ago. Isn't it time to promote them onwards?

loicalbertin and others added 5 commits October 21, 2016 15:53
…s defined as Java parameters (-DproxyHost -DproxyPort -Dhttp.proxy..., -Dhttp.nonProxyHosts, ...)
Before this commit, if you attempted to use the Stash pull-request
builder plug in with Jenkins Pipeline, you would encounter an error.
Ultimately, this error was due to the fact that this plug in require
jobs to use the AbstractProject base class, but the Jenkins Pipeline
job class (WorkflowJob) doesn't.

The good news is that WorkflowJob and AbstractProject share a base
class of Job, and that most functionality of this plug in can be made
available to WorkflowJob refactoring uses of AbstractProject to uses
of Job instead.  For cases where this does not provide enough
functionality, there are interfaces implemented in common by both
(ParameterizedJobMixIn.ParameterizedJob, Queue.Task) that fill the
gaps.

The one remaining piece of functionality this commit does not extend
to be available to Jenkins Pipeline is custom post-build comments, so
a TODO to that effect was added.
Update to Apache httpclient 4.5.2. This allows to use proxies settings
Add support for Jenkins Pipeline.
@jimklimov
Copy link
Author

Pinging @nemccarthy as the only maintainer, to get this PR moving forward :)

@jakub-bochenski
Copy link

I would rather review them and possibly integrate one by one.
Also I would rather first just push out a new release with only simple bugfixes like #26

@jakub-bochenski jakub-bochenski force-pushed the master branch 2 times, most recently from 9c4d199 to 84bbeb8 Compare January 25, 2019 11:08
@jimklimov
Copy link
Author

Makes sense to (update and) merge them separately...

@jakub-bochenski
Copy link

OK, closing this one then

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants