DrupalPod icon indicating copy to clipboard operation
DrupalPod copied to clipboard

Devel 5 required for Drupal 10

Open robertoperuzzo opened this issue 3 years ago • 5 comments

The Problem/Issue/Bug

During the workspace spinning up I get this error

Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - Root composer.json requires drupal/devel ^4.1 -> satisfiable by drupal/devel[4.1.0, ..., 4.x-dev].
    - drupal/devel[4.1.0, ..., 4.x-dev] require drupal/core ^8.8 || ^9 -> found drupal/core[8.8.0-alpha1, ..., 8.9.x-dev, 9.0.0-alpha1, ..., 9.5.x-dev] but the package is fixed to 10.0.0-alpha6 (lock file version) by a partial update and that version does not match. Make sure you list it as an argument for the update command.

Use the option --with-all-dependencies (-W) to allow upgrades, downgrades and removals for packages currently locked to specific versions.

Installation failed, reverting ./composer.json and ./composer.lock to their original content.
composer [require drush/drush:^11.0 drupal/coder:^8.3 drupal/devel:^4.1 drupal/admin_toolbar:^3.1] failed, composer command failed: exit status 2. stderr=

How this PR Solves The Problem

Install the 5.0.0-beta1 version of devel module.

Manual Testing Instructions

@shaal I need help to understand how can I test it because I am newbie on DrupalPod.

Related Issue Link(s)

#97

Release/Deployment notes

Closes #97

robertoperuzzo avatar Sep 14 '22 19:09 robertoperuzzo

CodeSandbox logoCodeSandbox logo  Open in CodeSandbox Web Editor | VS Code | VS Code Insiders

codesandbox[bot] avatar Sep 14 '22 19:09 codesandbox[bot]

👇 Click on the image for a new way to code review
  • Make big changes easier — review code in small groups of related files

  • Know where to start — see the whole change at a glance

  • Take a code tour — explore the change with an interactive tour

  • Make comments and review — all fully sync’ed with github

    Try it now!

Review these changes using an interactive CodeSee Map

Legend

CodeSee Map Legend

ghost avatar Sep 14 '22 19:09 ghost

Aside from devel 5 not being in beta anymore I think the workaround should be removed to be able to install devel in Drupal 10.1.x and just use devel 5 for Drupal 9 and 10.

Can you you make those changes so I can close my other PR related to this?

Also a better PR title that includes the word devel can help.

Thanks for bringing it up.

rodrigoaguilera avatar Sep 23 '22 12:09 rodrigoaguilera

@rodrigoaguilera title changed. I'm going to update this PR as soon as I have spare time. Thank you!

robertoperuzzo avatar Sep 23 '22 12:09 robertoperuzzo

:thinking: Now I'm wondering if we should maybe just let composer figure out the version instead of restricting so we don't have the same problem when devel 6 is released. What do you think?

rodrigoaguilera avatar Sep 23 '22 13:09 rodrigoaguilera