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

[SITE-3373] Stop scaffolding the settings.pantheon.php file. #34

Open
wants to merge 10 commits into
base: 11.x
Choose a base branch
from

Conversation

greg-1-anderson
Copy link
Member

@greg-1-anderson greg-1-anderson commented Jan 24, 2025

This PR allows sites to opt-in to a new method for Pantheon to deliver Drupal integrations to Drupal sites. In the future, we will automate the removal of the scaffolding technique by way of the Composer pre-update script in the Drupal Composer Managed upstream.

Note that the instructions in this PR won't work yet, but the following interim steps probably will (n.b. Drupal 11 / Drupal CMS sites only):

$ terminus connection:set sftp
$ terminus composer require pantheon-systems/drupal-integrations:dev-stop-scaffolding
$ terminus drush ev '\Pantheon\Integrations\Utils::stopScaffolding();'
$ terminus composer update
$ terminus env:commit --message "Stop scaffolding Pantheon's Drupal integrations, and include directly from vendor instead."

This should remove the error messages:

Unable to download modules via the UI: Any packages other than the implicitly allowed packages are not allowed to scaffold files. See the scaffold documentation for more information. pantheon-systems/drupal-integrations

and:

Your site cannot be automatically updated until further action is performed.
Any packages other than the implicitly allowed packages are not allowed to scaffold files. See the scaffold documentation for more information.
pantheon-systems/drupal-integrations

@greg-1-anderson greg-1-anderson requested a review from a team as a code owner January 24, 2025 20:56
@greg-1-anderson greg-1-anderson changed the base branch from 10.x to 11.x January 24, 2025 20:58
@greg-1-anderson greg-1-anderson requested a review from a team as a code owner January 24, 2025 20:58
pwtyler
pwtyler previously approved these changes Jan 24, 2025
@greg-1-anderson greg-1-anderson marked this pull request as draft January 24, 2025 21:53
@greg-1-anderson greg-1-anderson marked this pull request as ready for review January 24, 2025 23:43
@greg-1-anderson greg-1-anderson dismissed pwtyler’s stale review January 24, 2025 23:46

Implementation provided.

This was referenced Jan 30, 2025
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.

3 participants