-
Notifications
You must be signed in to change notification settings - Fork 396
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
DT-588: New projects use Composer Scaffold. #3944
Conversation
@@ -28,7 +28,6 @@ | |||
"dealerdirect/phpcodesniffer-composer-installer": "^0.5.0", | |||
"dflydev/dot-access-data": "^1.1.0", | |||
"doctrine/common": "^2.5", | |||
"doctrine/inflector": "~1.1.0", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This "dependency" only existed to support PHP 5.6, so it's safe to remove: #1829
@TravisCarden I have a conundrum. This PR is changing BLT's project template to use Composer Scaffold, which is only available in Drupal 8.8. Thus you cannot create Drupal 8.7 projects anymore. From a customer standpoint, I think that's fine (no one wants to create new projects on an old version of core.) But this causes ORCA's previous version test to choke. Any ideas how to solve this? To solve this in the spirit of ORCA, for the previous version test, it seems like you'd want to test that you can add BLT to an existing Drupal 8.7 project (which is possible), rather than testing that you can create a new Drupal 8.7 project with BLT (which is not possible). But that's a pretty big architectural change for ORCA. |
Well, @danepowell, ORCA's policy is to test all supported scenarios. (See, for example, how Lightning Core 8.x-3.x only runs a couple of jobs.) So if you can't create a new BLT site on Drupal 8.7, then you don't need to. On the other hand, as you point out, BLT has functions independent of new site creation that should continue to work and are still supported. If (as you will know better than I) you have a clean version break for core version support (e.g., 10.x supports |
Thanks. It's not ideal but I agree that just killing that test is the best approach for now. Done and done. |
"knplabs/github-api": "^2.6", | ||
"php-http/guzzle6-adapter": "^1.1", | ||
"phpunit/phpunit": "^4.8 || ^6.5", | ||
"squizlabs/php_codesniffer": "^3.0.1", | ||
"webflo/drupal-core-require-dev": "^8.6.0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🎉
Fixes #3563
Composer Scaffold replaces and deprecates the older Drupal Scaffold as of Drupal 8.8. However, the two packages conflict and since only Drupal Scaffold is available prior to 8.8, some accommodations have to be made. Namely, we have to disable the ORCA "previous Drupal release" test until Drupal 8.9 is released. This is fine since in real-world usage, customers are unlikely to be creating new Drupal 8.7 projects.
Note that this PR only affects new projects. Existing Drupal 8.7 users can still update to BLT 11, or add BLT 11 to their projects.
Changes proposed
acquia/composer-scaffold
instead ofdrupal-scaffold
, which pulls in Composer Scaffold.Steps to replicate the issue
cp -r subtree-splits/blt-project /tmp/
composer config --global repositories.blt-project path /tmp/blt-project
composer config --global repositories.blt path /home/dane/packages/blt
composer create-project --no-interaction acquia/blt-project:dev-master
composer config --global --unset repositories.blt && composer config --global --unset repositories.blt-project