Uploaded image for project: 'Jenkins'
  1. Jenkins
  2. JENKINS-64139

FSTrigger broken with tables to divs

XMLWordPrintable

    • fstrigger-plugin 0.40

      Reports from users in https://issues.jenkins-ci.org/browse/JENKINS-64072 that this plugin is broken

      I did some initial work updating the baseline so I could test it but there's a million spotbugs issues to fix...

          [JENKINS-64139] FSTrigger broken with tables to divs

          Raihaan Shouhell added a comment -

          I took a peek at this and after trying to get the plugin to compile with more modern tooling, animal sniffer would throw an error

          Raihaan Shouhell added a comment - I took a peek at this and after trying to get the plugin to compile with more modern tooling, animal sniffer would throw an error

          Tim Jacomb added a comment -

          it's because of the empty jar file, animal sniffer doesn't like it.

          I deleted it to get past that error and got a million spotbugs error

          Here's my branch:
          https://github.com/jenkinsci/fstrigger-plugin/compare/master...timja:tables-to-divs?expand=1

          Tim Jacomb added a comment - it's because of the empty jar file, animal sniffer doesn't like it. I deleted it to get past that error and got a million spotbugs error Here's my branch: https://github.com/jenkinsci/fstrigger-plugin/compare/master...timja:tables-to-divs?expand=1

          Raihaan Shouhell added a comment -

          Thanks for the tip, I've got a working PR. But its quite complicated due to the tooling updates and spotbugs fixes :/

          Raihaan Shouhell added a comment - Thanks for the tip, I've got a working PR. But its quite complicated due to the tooling updates and spotbugs fixes :/

          Tony Noble added a comment -

          Apologies for the lack of response so far - sorting out this plugin has been on my work list for a while (and more urgently since the urltrigger issue reared its head)

          I'll take a look at the PR and get a realease out ASAP.

          Tony Noble added a comment - Apologies for the lack of response so far - sorting out this plugin has been on my work list for a while (and more urgently since the urltrigger issue reared its head) I'll take a look at the PR and get a realease out ASAP.

          Chris Beswick added a comment -

          timja Thanks for raising this issue. From JENKINS-64072 you said:

          Let us know on the issue if you need it fixed otherwise I'm going to look at more important plugins first

           For me this plugin was not used and will have no requirement to reinstall it so its not important to me.

           

          Chris Beswick added a comment - timja Thanks for raising this issue. From  JENKINS-64072 you said: Let us know on the issue if you need it fixed otherwise I'm going to look at more important plugins first  For me this plugin was not used and will have no requirement to reinstall it so its not important to me.  

          Tony Noble added a comment -

          Release 0.40 of fstrigger should resolve this issue

          Tony Noble added a comment - Release 0.40 of fstrigger should resolve this issue

            raihaan Raihaan Shouhell
            timja Tim Jacomb
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

              Created:
              Updated:
              Resolved: