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 composer.json for Drupal 8.8 #96

Merged
merged 2 commits into from
Nov 27, 2019
Merged

Conversation

becw
Copy link
Member

@becw becw commented Nov 4, 2019

Updates the composer.json to use the new drupal/core-composer-scaffold package.

@see slides from Drupalcon Amsterdam and the drupal/core-composer-scaffold README.

This new scaffolding tool doesn't allow you to manage scaffolding files from your dependencies beyond saying "yes", "no", or "append". So, for the .htaccess file, which we want Drupal core to install but not update, we have to reference its full path as if it belongs to our project.

There's a Drupal core issue on the scaffolding behavior, which allows Drupal core to predictably handle user modifications to scaffolded files: https://www.drupal.org/project/drupal/issues/3092563. It looks like this fix will NOT be included in the 8.8.0 release, which means that we should manage the .htaccess file ourselves for the time being.

becw added 2 commits November 26, 2019 17:35
…it is broken -- drupal/core-composer-scaffold seems to be looking in palantirnet/drupal-skeleton for the .htaccess file.
…port managing dependencies' scaffolding files.
@becw becw force-pushed the composer-changes-for-drupal-8.8 branch from 7afaf92 to 7963abb Compare November 26, 2019 23:36
@becw becw merged commit 75b0c41 into drupal8 Nov 27, 2019
@becw becw deleted the composer-changes-for-drupal-8.8 branch November 27, 2019 23:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants