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

FSTrigger broken with tables to divs

    XMLWordPrintable

    Details

    • Similar Issues:
    • Released As:
      fstrigger-plugin 0.40

      Description

      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...

        Attachments

          Issue Links

            Activity

            Hide
            raihaan 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

            Show
            raihaan 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
            Hide
            timja 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

            Show
            timja 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
            Hide
            raihaan 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 :/

            Show
            raihaan 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 :/
            Hide
            stealthdj 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.

            Show
            stealthdj 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.
            Hide
            javadevil Chris Beswick added a comment -

            Tim Jacomb 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.

             

            Show
            javadevil Chris Beswick added a comment - Tim Jacomb 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.  
            Hide
            stealthdj Tony Noble added a comment -

            Release 0.40 of fstrigger should resolve this issue

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

              People

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

                Dates

                Created:
                Updated:
                Resolved: