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

          Tim Jacomb created issue -
          Tim Jacomb made changes -
          Labels New: tables-to-divs-regression
          Tim Jacomb made changes -
          Link New: This issue is related to JENKINS-64072 [ JENKINS-64072 ]

          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 made changes -
          Assignee Original: Tony Noble [ stealthdj ] New: Raihaan Shouhell [ raihaan ]
          Raihaan Shouhell made changes -
          Status Original: Open [ 1 ] New: In Progress [ 3 ]
          Raihaan Shouhell made changes -
          Status Original: In Progress [ 3 ] New: In Review [ 10005 ]
          Raihaan Shouhell made changes -
          Remote Link New: This issue links to "fstrigger-plugin#10 (Web Link)" [ 26216 ]

          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 :/

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

              Created:
              Updated:
              Resolved: