Hi Tony, We incorporated much of the feedback by Oleg (thanks again for the review!), but yes, I think there are still some unprocessed comments (those without +1 marker). We need to decide how urgent they are - currently we are not working on them due to capacity reasons.
Nevertheless you need to know we are using this version of the plugin productively on more than hundred (team) Jenkinses in our company. Identified bugs in the last 2 years have been fixed, this most likely includes bugs which are still contained in the current official version of the plugin. Kind regards, Alex Am Dienstag, 24. April 2018 13:45:04 UTC+2 schrieb Tony Noble: > > I did spend some time looking at that PR, but it appears as though there > is still work to be done on it, or at least, changes requested. > > Is that not the case? I got a little lost trying to figure out what was > still to be done. > > Tony > > On Tue, Apr 24, 2018 at 12:18 PM, 'Alexander Link' via Jenkins Developers > <[email protected] <javascript:>> wrote: > >> Hi, >> >> >> >> as Oleg mentioned we did many changes to this plugin in the last 2 years >> – in our fork unfortunately, since contribution did not work due to missing >> maintainers. >> >> You find *all* changes in our openSource branch: >> https://github.com/sap-production/parameterized-remote-trigger-plugin/tree/openSource >> >> >> >> >> We don’t have enough resources to become maintainers ourselves, but we >> are happy to support you with this plugin. >> >> >> >> My recommendation would be to decide on this PR first to avoid later >> conflicts: >> https://github.com/jenkinsci/parameterized-remote-trigger-plugin/pull/32 >> >> >> >> Kind regards, Alex >> >> >> >> PS: Here some additional comments about the Pull Request: >> >> >> https://github.com/jenkinsci/parameterized-remote-trigger-plugin/pull/32#issuecomment-373000207 >> >> >> >> https://github.com/jenkinsci/parameterized-remote-trigger-plugin/pull/32#issuecomment-338175831 >> >> >> >> >> >> Am Dienstag, 24. April 2018 10:31:16 UTC+2 schrieb Oleg Nenashev: >>> >>> I would like to point out that there is a major work being done on >>> Pipeline compatibility in >>> https://github.com/jenkinsci/parameterized-remote-trigger-plugin/pull/32 >>> . It would be great to invite Alejandra Ferreiro Vidal and Alexander Link >>> to this discussion. Will do that. >>> >>> BR, Oleg >>> >>> On Monday, April 23, 2018 at 8:13:54 AM UTC+2, Kai-Hsiang Chang wrote: >>>> >>>> @Tony, thx for the cc. >>>> >>>> github id: cashlalala >>>> Jenkins JIRA id: cashlalala >>>> >>>> Tony Noble於 2018年4月20日星期五 UTC+8下午8時41分52秒寫道： >>>>> >>>>> I'll happily co-maintain if required. There's a PR needing to be >>>>> merged to make it work with CSRF protection (that I've suggested a minor >>>>> change to) that really could do with being applied, amongst others. >>>>> >>>>> Christoph shows as last person to make any git commits, so I've cc'd >>>>> him. >>>>> >>>>> Tony >>>>> >>>>> github id: TonyNoble >>>>> Jenkins JIRA id: stealthdj >>>>> >>>>> On Fri, Apr 20, 2018 at 9:11 AM, Daniel Beck <[email protected]> wrote: >>>>> >>>>>> >>>>>> > On 20. Apr 2018, at 03:51, Kai-Hsiang Chang <[email protected]> >>>>>> wrote: >>>>>> > >>>>>> > this plugin hasn't been updated for at least 2 years, >>>>>> > >>>>>> > I'd like to ask the permission to maintain the plugin. >>>>>> > >>>>>> > please help me with that. >>>>>> > >>>>>> >>>>>> Please follow the instructions from >>>>>> https://wiki.jenkins.io/display/JENKINS/Adopt+a+Plugin#AdoptaPlugin-IknowwhichpluginIwanttohelpwith,whatshouldIdonow >>>>>> ? >>>>>> >>>>>> -- >>>>>> You received this message because you are subscribed to the Google >>>>>> Groups "Jenkins Developers" group. >>>>>> To unsubscribe from this group and stop receiving emails from it, >>>>>> send an email to [email protected]. >>>>>> To view this discussion on the web visit >>>>>> https://groups.google.com/d/msgid/jenkinsci-dev/EA80C67F-95F0-4E4C-A83C-4451B8439B56%40beckweb.net >>>>>> . >>>>>> For more options, visit https://groups.google.com/d/optout. >>>>>> >>>>> >>>>> -- >> You received this message because you are subscribed to the Google Groups >> "Jenkins Developers" group. >> To unsubscribe from this group and stop receiving emails from it, send an >> email to [email protected] <javascript:>. >> To view this discussion on the web visit >> https://groups.google.com/d/msgid/jenkinsci-dev/372bf671-47a6-4fa1-ae10-132599a3ca28%40googlegroups.com >> >> <https://groups.google.com/d/msgid/jenkinsci-dev/372bf671-47a6-4fa1-ae10-132599a3ca28%40googlegroups.com?utm_medium=email&utm_source=footer> >> . >> >> For more options, visit https://groups.google.com/d/optout. >> > > -- You received this message because you are subscribed to the Google Groups "Jenkins Developers" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To view this discussion on the web visit https://groups.google.com/d/msgid/jenkinsci-dev/39618b89-3d24-4956-b571-19da5bbfb1d6%40googlegroups.com. For more options, visit https://groups.google.com/d/optout.
