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

Building / Checking evergreen modifies the npm dependencies

XMLWordPrintable

    • Icon: Task Task
    • Resolution: Fixed
    • Icon: Minor Minor
    • evergreen
    • None
    • Evergreen - Milestone 2

      When running make check on a fresh clone of Evergreen, it is possible to see modifications on package-lock.json files. I could see that on distribution/client/package-lock.json. See the patch attached. Modifications were minor but it could lead to more issues when transitive dependencies are upgraded or such.

      Making sure to use the correct dependencies could prevent "invalid" development or not intended dependencies modifications in the project.

        1. evergreen.patch
          3 kB

          [JENKINS-54204] Building / Checking evergreen modifies the npm dependencies

          Adrien Lecharpentier created issue -
          Baptiste Mathus made changes -
          Assignee Original: R. Tyler Croy [ rtyler ] New: Adrien Lecharpentier [ alecharp ]
          Baptiste Mathus made changes -
          Sprint New: Evergreen - Milestone 2 [ 516 ]
          Baptiste Mathus made changes -
          Status Original: Open [ 1 ] New: In Progress [ 3 ]
          Baptiste Mathus made changes -
          Summary Original: Building / Checking evergreen modify the npm dependencies New: Building / Checking evergreen modifies the npm dependencies
          Baptiste Mathus made changes -
          Status Original: In Progress [ 3 ] New: In Review [ 10005 ]
          Baptiste Mathus made changes -
          Remote Link New: This issue links to "evergreen PR (Web Link)" [ 21957 ]

          Baptiste Mathus added a comment -

          thanks alecharp !

          Baptiste Mathus added a comment - thanks alecharp !
          Baptiste Mathus made changes -
          Resolution New: Fixed [ 1 ]
          Status Original: In Review [ 10005 ] New: Closed [ 6 ]
          CC bomber made changes -
          Resolution Original: Fixed [ 1 ]
          Status Original: Closed [ 6 ] New: In Review [ 10005 ]

            naveenboni Naveen Boni
            alecharp Adrien Lecharpentier
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: