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

Replace FlowNodeNavigator with FlowGraphWalker

    XMLWordPrintable

    Details

    • Similar Issues:

      Description

      FlowNodeNavigator was created without knowing the existence of FlowGraphWalker. FlowGraphWalker is the official API and is a better impl, so would be good to eliminate FlowNodeNavigator.

        Attachments

          Activity

          Show
          tfennelly Tom FENNELLY added a comment - PR: https://github.com/jenkinsci/pipeline-stage-view-plugin/pull/2
          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: tfennelly
          Path:
          rest-api/src/main/java/com/cloudbees/workflow/flownode/FlowNodeNavigator.java
          rest-api/src/main/java/com/cloudbees/workflow/flownode/FlowNodeUtil.java
          rest-api/src/main/java/com/cloudbees/workflow/rest/external/RunExt.java
          http://jenkins-ci.org/commit/pipeline-stage-view-plugin/59eed49fe73b6f506f67a50575f538cf61e5e5bc
          Log:
          [Fix JENKINS-33486] Replace FlowNodeNavigator with FlowGraphWalker

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: tfennelly Path: rest-api/src/main/java/com/cloudbees/workflow/flownode/FlowNodeNavigator.java rest-api/src/main/java/com/cloudbees/workflow/flownode/FlowNodeUtil.java rest-api/src/main/java/com/cloudbees/workflow/rest/external/RunExt.java http://jenkins-ci.org/commit/pipeline-stage-view-plugin/59eed49fe73b6f506f67a50575f538cf61e5e5bc Log: [Fix JENKINS-33486] Replace FlowNodeNavigator with FlowGraphWalker
          Hide
          scm_issue_link SCM/JIRA link daemon added a comment -

          Code changed in jenkins
          User: Sam Van Oort
          Path:
          rest-api/src/main/java/com/cloudbees/workflow/flownode/FlowNodeNavigator.java
          rest-api/src/main/java/com/cloudbees/workflow/flownode/FlowNodeUtil.java
          rest-api/src/main/java/com/cloudbees/workflow/rest/external/RunExt.java
          http://jenkins-ci.org/commit/pipeline-stage-view-plugin/e99d804ebbe4773debf7ba359895e5ed56daec05
          Log:
          Merge pull request #2 from tfennelly/JENKINS-33486

          [Fix JENKINS-33486] Replace FlowNodeNavigator with FlowGraphWalker

          Compare: https://github.com/jenkinsci/pipeline-stage-view-plugin/compare/37e3c1c4541b...e99d804ebbe4

          Show
          scm_issue_link SCM/JIRA link daemon added a comment - Code changed in jenkins User: Sam Van Oort Path: rest-api/src/main/java/com/cloudbees/workflow/flownode/FlowNodeNavigator.java rest-api/src/main/java/com/cloudbees/workflow/flownode/FlowNodeUtil.java rest-api/src/main/java/com/cloudbees/workflow/rest/external/RunExt.java http://jenkins-ci.org/commit/pipeline-stage-view-plugin/e99d804ebbe4773debf7ba359895e5ed56daec05 Log: Merge pull request #2 from tfennelly/ JENKINS-33486 [Fix JENKINS-33486] Replace FlowNodeNavigator with FlowGraphWalker Compare: https://github.com/jenkinsci/pipeline-stage-view-plugin/compare/37e3c1c4541b...e99d804ebbe4
          Hide
          svanoort Sam Van Oort added a comment -

          verified & released.

          Show
          svanoort Sam Van Oort added a comment - verified & released.

            People

            • Assignee:
              tfennelly Tom FENNELLY
              Reporter:
              tfennelly Tom FENNELLY
            • Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: