I don't see why the hudson/Main.java class is still in core, it should either be deleted or moved to the External Monitor Job Type plugin.

          [JENKINS-70684] Remove Main.java from core

          Jesse Glick added a comment -

          https://github.com/jenkinsci/external-monitor-job-plugin/blob/683c09d993b9e9da5f883f4d0696eb45af323b55/src/main/java/hudson/cli/SetExternalBuildResultCommand.java#L12-L17 accomplishes the same thing in a more standard way. Removing hudson.Main would be incompatible for anyone still using the deprecated way. I doubt there are many people actively using the plugin at all, so I think it would be fine to just delete it and add a note to the upgrade guide.

          Jesse Glick added a comment - https://github.com/jenkinsci/external-monitor-job-plugin/blob/683c09d993b9e9da5f883f4d0696eb45af323b55/src/main/java/hudson/cli/SetExternalBuildResultCommand.java#L12-L17 accomplishes the same thing in a more standard way. Removing hudson.Main would be incompatible for anyone still using the deprecated way. I doubt there are many people actively using the plugin at all, so I think it would be fine to just delete it and add a note to the upgrade guide.

          James Nord added a comment -

          isDevelopment and isUnitTest are still used and plugins would need adapting?

           

          James Nord added a comment - isDevelopment and isUnitTest are still used and plugins would need adapting?  

          Jesse Glick added a comment -

          Right, those two fields could stay. Everything else should be deleted.

          Jesse Glick added a comment - Right, those two fields could stay. Everything else should be deleted.

            Unassigned Unassigned
            kevingrdj Kevin Guerroudj
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated: