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

Remove unneeded dependencies (markdown, redux)

XMLWordPrintable

    • Blue Ocean 1.3, Blue Ocean 1.4 - beta 1, Blue Ocean 1.4 - beta 3, Blue Ocean 1.4 - beta 2, Blue Ocean 1.4 - beta 5

      Once karaoke has moved off redux, and personalisation, redux may be removed as a dependency saving download and eval time. 

      Ditty for markdown (which cliffmeyers mentioned may not be needed now with the new empty states, but correct me if wrong). 

      Well worth trimming things we know aren't needed. 

          [JENKINS-43211] Remove unneeded dependencies (markdown, redux)

          Cliff Meyers added a comment -

          To eliminate redux, following areas need to be ported:

          • blueocean-personalization (full module)
          • blueocean-web (all bootstrapping code for stores, middleware, etc)
          • blueocean-dashboard
            • Dashboard
            • TestResults
            • various tests / specs

          To eliminate "react-remarkable", following areas need porting:

          • All remaining empty states (at present, only for RunDetailsArtifacts)
          • ParametersRunButton
          • inputStep

          Cliff Meyers added a comment - To eliminate redux, following areas need to be ported: blueocean-personalization (full module) blueocean-web (all bootstrapping code for stores, middleware, etc) blueocean-dashboard Dashboard TestResults various tests / specs To eliminate "react-remarkable", following areas need porting: All remaining empty states (at present, only for RunDetailsArtifacts) ParametersRunButton inputStep

          Michael Neale added a comment -

          cliffmeyers so based on that - my finger int he air guess is probably a good few days solid work to a week to flush it all out? If so - no hurry for this as "minor". Low cost-benefit return. 

          Michael Neale added a comment - cliffmeyers so based on that - my finger int he air guess is probably a good few days solid work to a week to flush it all out? If so - no hurry for this as "minor". Low cost-benefit return. 

          Cliff Meyers added a comment -

          Since I am likely to add a new dep in JENKINS-38323 I think it's time to remove some old ones

          Cliff Meyers added a comment - Since I am likely to add a new dep in JENKINS-38323 I think it's time to remove some old ones

          Cliff Meyers added a comment -

          Added link to WIP PR.

          Cliff Meyers added a comment - Added link to WIP PR.

          Cliff Meyers added a comment -

          This work was completed and ran fine locally but causes 100% failure of ATH in CI (but only in CI). This is effectively on-hold for now, refer to comments in PR for details.

          Cliff Meyers added a comment - This work was completed and ran fine locally but causes 100% failure of ATH in CI (but only in CI). This is effectively on-hold for now, refer to comments in PR for details.

          Karl Shultz added a comment -

          Testing Notes:

          • There are already new tests in the PR, which are being run and discussed
          • Automated testing is already catching problems that this change might introduce, which is great.

          Karl Shultz added a comment - Testing Notes: There are already new tests in the PR, which are being run and discussed Automated testing is already catching problems that this change might introduce, which is great.

          Cliff Meyers added a comment -

          Added link to the new / latest PR that should allow this be merged: https://github.com/jenkinsci/blueocean-plugin/pull/1627

          Cliff Meyers added a comment - Added link to the new / latest PR that should allow this be merged: https://github.com/jenkinsci/blueocean-plugin/pull/1627

            cliffmeyers Cliff Meyers
            michaelneale Michael Neale
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:
              Resolved: