Hi Daniel, it seems no response from the owner, maybe we can start the process?
Alexander Link於 2018年4月27日星期五 UTC+8下午2時55分19秒寫道: > > Hi, > > I collected all open review comments: > https://github.com/sap-production/parameterized-remote-trigger-plugin/issues/20 > We will check the urgency and process the urgent ones. > > But in general we are using the plugin productively and never had severe > issues. Minor bugs (recognized in our scenarios) have been fixed. > I would also vote for merging this PR first. We already tried to split it > up into single meaningful PRs but this is almost not possible anymore. > > It seems you will be the maintainer in future, so you have to decide if > the changes are good enough :) > > Thanks a lot and kind regards, > Alex > > Am Dienstag, 24. April 2018 15:45:22 UTC+2 schrieb Kai-Hsiang Chang: >> >> Hi Alex, >> >> If this version has been tested & run for 2 years in your company, I >> think it's okay to just apply this huge PR and refine the code quality & >> the comments later. >> But this is just my opinion, I'm not the maintainer. >> >> 2018-04-24 20:22 GMT+08:00 'Alexander Link' via Jenkins Developers < >> [email protected]>: >> >>> 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]> 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]. >>>>> 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 a topic in the >>> Google Groups "Jenkins Developers" group. >>> To unsubscribe from this topic, visit >>> https://groups.google.com/d/topic/jenkinsci-dev/y1Sv9MQhyaY/unsubscribe. >>> To unsubscribe from this group and all its topics, 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 >>> >>> <https://groups.google.com/d/msgid/jenkinsci-dev/39618b89-3d24-4956-b571-19da5bbfb1d6%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/e820cc7f-1e91-442f-8206-c78efa05c9ee%40googlegroups.com. For more options, visit https://groups.google.com/d/optout.
