Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-38532

Fix composer dependencies versions when master branch becomes STABLE

    Details

      Description

      As commented in MDL-38524 working with non-fixed behat dependencies can be complicated for maintenance reasons as there are different upstream project requiring different versions of different dependencies and changes can affect tests results. We can use wildcards (the latest version in composer nomenclature, 1.2.* for example) during DEV.

      This change will be tied to changes in moodlehq/behatextension, we can begin with 1.26.* for 2.6 and switch the wildcards to fixed versions in 1.25.lastone for 2.5

        Gliffy Diagrams

          Attachments

            Issue Links

              Activity

              Hide
              damyon Damyon Wiese added a comment -

              Is now a good time to freeze this David?

              Show
              damyon Damyon Wiese added a comment - Is now a good time to freeze this David?
              Hide
              damyon Damyon Wiese added a comment -

              Also we need to decide if we want to remove the change in MDL-38878 for stable branches when we do this (checking for composer updates can be so slow).

              Show
              damyon Damyon Wiese added a comment - Also we need to decide if we want to remove the change in MDL-38878 for stable branches when we do this (checking for composer updates can be so slow).
              Hide
              dmonllao David Monllaó added a comment -

              It makes sense, thanks I didn't thought of it, I changed the fix version to 2.5 and I will work on it just after 2.5 is released.

              Show
              dmonllao David Monllaó added a comment - It makes sense, thanks I didn't thought of it, I changed the fix version to 2.5 and I will work on it just after 2.5 is released.
              Hide
              dmonllao David Monllaó added a comment -

              As commented in https://tracker.moodle.org/browse/MDL-38041#comment-220094 we can use 1.25.x for Moodle 2.5, 1.26.x for Moodle 2.6...

              Show
              dmonllao David Monllaó added a comment - As commented in https://tracker.moodle.org/browse/MDL-38041#comment-220094 we can use 1.25.x for Moodle 2.5, 1.26.x for Moodle 2.6...
              Hide
              dmonllao David Monllaó added a comment -

              Adding pull branches, but waiting to send it to review as it should be integrated after the release of 2.5.

              Step added to http://docs.moodle.org/dev/Major_release_process#Post_release

              Show
              dmonllao David Monllaó added a comment - Adding pull branches, but waiting to send it to review as it should be integrated after the release of 2.5. Step added to http://docs.moodle.org/dev/Major_release_process#Post_release
              Hide
              dmonllao David Monllaó added a comment -

              I have doubts about removing the testing_update_composer_dependencies() when updating the test site, any feedback about it?

              Show
              dmonllao David Monllaó added a comment - I have doubts about removing the testing_update_composer_dependencies() when updating the test site, any feedback about it?
              Hide
              dmonllao David Monllaó added a comment -

              Submitting for integration as talked with Eloy

              Show
              dmonllao David Monllaó added a comment - Submitting for integration as talked with Eloy
              Hide
              dmonllao David Monllaó added a comment -

              I'm removing the removal of the composer update, we can also have updates of the 1.25.x branch if new bugs are detected and the easy-to-do installation should manage this changes, also the checking of the dependencies versions is very short comparing to the required time to install the site. The alternative is to add a --skipcomposer option

              Show
              dmonllao David Monllaó added a comment - I'm removing the removal of the composer update, we can also have updates of the 1.25.x branch if new bugs are detected and the easy-to-do installation should manage this changes, also the checking of the dependencies versions is very short comparing to the required time to install the site. The alternative is to add a --skipcomposer option
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              (added an extra commit bumping version)

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - (added an extra commit bumping version)
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              Passing, components were installed/upgraded ok.

              Side note: After discussing it with David, it seems that the "unique-branch" strategy in the behat repo is not going to be enough and we need branching there too (ideally, like we branch moodle). That way each branch can have its own evolution if needed.

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - Passing, components were installed/upgraded ok. Side note: After discussing it with David, it seems that the "unique-branch" strategy in the behat repo is not going to be enough and we need branching there too (ideally, like we branch moodle). That way each branch can have its own evolution if needed.
              Hide
              dmonllao David Monllaó added a comment -

              Confirmed that composer also gets tags from other branches than master so there will be no problem with this approach

              Show
              dmonllao David Monllaó added a comment - Confirmed that composer also gets tags from other branches than master so there will be no problem with this approach
              Hide
              stronk7 Eloy Lafuente (stronk7) added a comment -

              Did you think this day was not going to arrive ever?

              Your patience has been rewarded, yay, sent upstream, thanks!

              Closing...ciao

              Show
              stronk7 Eloy Lafuente (stronk7) added a comment - Did you think this day was not going to arrive ever? Your patience has been rewarded, yay, sent upstream, thanks! Closing...ciao

                People

                • Votes:
                  0 Vote for this issue
                  Watchers:
                  3 Start watching this issue

                  Dates

                  • Created:
                    Updated:
                    Resolved:
                    Fix Release Date:
                    14/May/13