BLT-4677: Add EnvironmentDetector for acquia cloud IDE.

acquia / blt

Acquia's toolset for automating Drupal 8 and 9 development, testing, and deployment.

https://docs.acquia.com/blt/

GNU General Public License v2.0

442 stars 394 forks source link

Closed shriniyesare closed 1 year ago

shriniyesare commented 1 year ago

Motivation Issue link: https://github.com/acquia/blt/issues/4677 BLT-5213: local.settings.php is not detecting on Acquia cloud IDE.

Fixes #4677

Proposed changes Adds a check to IDE Environment Detector and set path to local.settings.php file.

Testing steps

Create an IDE on Acquia cloud.
Setup the project on Acquia cloud with BLT.
Make changes to local.settings.php. Example change the database settings or add site studio keys.
Verify the changes reflect.

danepowell commented 1 year ago

This behavior is by design for the moment. We haven't found a good solution for dealing with Cloud-based IDEs: https://github.com/acquia/blt/issues/4175

mikemadison13 commented 1 year ago

There is a plugin that handles this, https://github.com/pavlosdan/blt-ide-local. this is how i've handled this in the past.

danepowell commented 1 year ago

I welcome ideas on how to solve this but it needs to at least be backwards-compatible, or significant enough to warrant a new major release.