The documentation for this project includes a section on disabling the scaffolding for pantheon.upstream.yml at https://github.com/pantheon-systems/drupal-integrations#special-instructions-for-upstreams

However, commit ba2317c removed the pantheon.upstream.yml from this package.

Assuming the intent is that this package is not responsible for scaffolding the pantheon.upstream.yml (or, per 7649ccf, the pantheon.yml), the documentation referenced above can be removed.

https://github.com/pantheon-systems/drupal-integrations/blob/10.x/composer.json#L7

And yet the pantheon instructions for migrating from drupal 9 to druapl 10 tell us to install version 10 of this package, which doesn't work: https://docs.pantheon.io/drupal-10#create-a-drupal-10-site

...
composer require --no-update pantheon-systems/drupal-integrations:^10
...

After installing a Drupal on a Pantheon site using a custom distribution that is using the scaffold files from this repo, I see the following error on the Drupal status report page:

Errors found: Configuration sync directory

The directory ../config does not exist.

As well as the following error in the Drupal watchdog logs:

Security warning: Couldn't write .htaccess file. Please create a .htaccess file in your ../config
directory which contains the following lines:

# Deny all requests from Apache 2.4+.
<IfModule mod_authz_core.c>
  Require all denied
</IfModule>

# Deny all requests from Apache 2.0-2.2.
<IfModule !mod_authz_core.c>
  Deny from all
</IfModule>

# Turn off all options we don't need.
Options -Indexes -ExecCGI -Includes -MultiViews

# Set the catch-all handler to prevent scripts from being executed.
SetHandler Drupal_Security_Do_Not_Remove_See_SA_2006_006
<Files *>
  # Override the handler again if we're run later in the evaluation list.
  SetHandler Drupal_Security_Do_Not_Remove_See_SA_2013_003
</Files>

# If we know how to do it safely, disable the PHP engine entirely.
<IfModule mod_php7.c>
  php_flag engine off
</IfModule>

Hi there!

We should be adding a config directory path to our settings.php files, so they work on local environments, correct? Or am I missing something in the latest versions of scaffolding files, and this isn't needed anymore?

(I was looking at https://github.com/pantheon-systems/drupal-integrations/blob/master/assets/initial.settings.php -- not saying you should necessarily add it there, mayyyyybe a code comment about it, but I understand if that's beyond the intended scope of this file -- I just wanted to see if I"m understanding things correctly.)

Currently, the provided settings.pantheon.php overrides $settings['config_sync_directory'] no matter what.

We're using the BLT-provided "settings.php parts" that configure the config directory according to blt.yml config file before settings.pantheon.php gets included.

I'd suggest we just wrap this with a if (empty($settings['config_sync_directory'])).

Drupal\Component\Plugin\Exception\PluginNotFoundException: The "view" entity type does not exist. in Drupal\Core\Entity\EntityTypeManager->getDefinition() (line 143 of core/lib/Drupal/Core/Entity/EntityTypeManager.php).

See the full bug here:

https://www.drupal.org/project/drupal/issues/3255318#comment-14345268

On by default?