DX-5800: Acquia\Blt\Robo\Common\EnvironmentDetector::getSiteName($site_path) must be compatible with Acquia\DrupalEnvironmentDetector\AcquiaDrupalEnvironmentDetector::getSiteName(string $site_path): ?string in /app/vendor/acquia/blt/src/Robo/Common/EnvironmentDetector.php

[bmartinez287]

Changes made to https://github.com/acquia/drupal-environment-detector require for this to be updated:

• Acquia\Blt\Robo\Common\EnvironmentDetector::getSiteName($site_path)
• Acquia\Blt\Robo\Common\EnvironmentDetector::isLocalEnv()

PHP Fatal error: Declaration of Acquia\Blt\Robo\Common\EnvironmentDetector::getSiteName($site_path) must be compatible with Acquia\DrupalEnvironmentDetector\AcquiaDrupalEnvironmentDetector::getSiteName(string $site_path): ?string in /app/vendor/acquia/blt/src/Robo/Common/EnvironmentDetector.php on line 236

Fatal error: Declaration of Acquia\Blt\Robo\Common\EnvironmentDetector::getSiteName($site_path) must be compatible with Acquia\DrupalEnvironmentDetector\AcquiaDrupalEnvironmentDetector::getSiteName(string $site_path): ?string in /app/vendor/acquia/blt/src/Robo/Common/EnvironmentDetector.php on line 236
[warning] Drush command terminated abnormally.

PHP Fatal error: Declaration of Acquia\Blt\Robo\Common\EnvironmentDetector::isLocalEnv() must be compatible with Acquia\DrupalEnvironmentDetector\AcquiaDrupalEnvironmentDetector::isLocalEnv(): bool in /app/vendor/acquia/blt/src/Robo/Common/EnvironmentDetector.php on line 105

Fatal error: Declaration of Acquia\Blt\Robo\Common\EnvironmentDetector::isLocalEnv() must be compatible with Acquia\DrupalEnvironmentDetector\AcquiaDrupalEnvironmentDetector::isLocalEnv(): bool in /app/vendor/acquia/blt/src/Robo/Common/EnvironmentDetector.php on line 105
[warning] Drush command terminated abnormally.

[mikemadison13]

Hi @bmartinez287 what version of BLT are you running?

[bmartinez287]

I was running 13.3 because we had some issues with 13.4. But I just saw this is fixed on 13.5.
That said can we merge this acquia/blt-travis#4? That came up after. Thanks for the quick feedback @mikemadison13

[joejoseph00]

@bmartinez287 I just hit this because the acquia documentation is outdated

https://docs.acquia.com/blt/install/adding-to-project/

Could you please update the acquia documentation to read ^13 instead of ^13.0 ?

see link ^^^

[joejoseph00]

#4674