Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

update to drupal 11 #27

Open
wants to merge 11 commits into
base: 11-0
Choose a base branch
from
Open

update to drupal 11 #27

wants to merge 11 commits into from

Conversation

byrond
Copy link
Contributor

@byrond byrond commented Dec 6, 2024

No description provided.

@agentrickard
Copy link
Contributor

@byrond I already have some of those changes in an 11-0 branch. We should reconcile the two.

@agentrickard agentrickard changed the base branch from main to 11-0 December 9, 2024 15:04
# is returned as 1.22.22 when Yarn starts, before switching directories.
cd $DDEV_DOCROOT/core
yarn install
cd -
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@byrond This change suggests a fail elsewhere. How do I trigger that failure?

Copy link
Contributor Author

@byrond byrond Dec 13, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This happened during ddev install:

yarn install v1.22.22
[1/5] Validating package.json...
error Drupal@: The engine "yarn" is incompatible with this module. Expected version ">= 4.1.1". Got "1.22.22"
error Found incompatible module.
info Visit https://yarnpkg.com/en/docs/cli/install for documentation about this command.

I believe it's because the required Yarn version isn't known until changing into the core directory, after Yarn has already started with the global version installed in the container.

The later version of Yarn is required now since I changed DDEV's nodejs_version to 22 as is now required by Drupal 11. The minimum version is 20, but I updated to the current LTS.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants