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

BLT-5201: Errors after upgrading to 13.7 #4653

Open
jedgar1mx opened this issue Feb 6, 2023 · 2 comments
Open

BLT-5201: Errors after upgrading to 13.7 #4653

jedgar1mx opened this issue Feb 6, 2023 · 2 comments
Labels
Bug Something isn't working

Comments

@jedgar1mx
Copy link

Describe the bug
I'm trying to upgrade blt to the latest version. I'm still stuck with 13.5.1 due to the changes in drupal-environment-detector. I'm using acquia/blt-travis to run tests on travis-ci. After switching to the master branch, travis-ci seems to work correctly with the latest blt. However, I now get issues with acquia-pipelines.

Expected behavior
Test should run successfully on both travis-ci and acquia-pipelines with latest BLT version.

Detailed error output
`[info] Drush bootstrap phase: bootstrapDrupalConfiguration()

TypeError: Acquia\BltTravis\Blt\Plugin\EnvironmentDetector\TravisDetector::getCiSettingsFile(): Return value must be of type string, none returned in Acquia\BltTravis\Blt\Plugin\EnvironmentDetector\TravisDetector::getCiSettingsFile() (line 16 of /mnt/tmp/local.prod/source/vendor/acquia/blt-travis/src/Blt/Plugin/EnvironmentDetector/TravisDetector.php).

[warning] Drush command terminated abnormally.

[Acquia\Blt\Robo\Tasks\DrushTask] Exit code 1 Time 0.19s

[error] Failed to install Drupal!
For troubleshooting guidance and support, see https://docs.acquia.com/blt/support/ `

@jedgar1mx jedgar1mx added the Bug Something isn't working label Feb 6, 2023
@github-actions github-actions bot changed the title Errors after upgrading to 13.7 BLT-5201: Errors after upgrading to 13.7 Feb 6, 2023
@mikemadison13
Copy link
Contributor

what version of the blt-travis plugin do you have? theoretically https://github.com/acquia/blt-travis/pull/5/files fixed this issue but maybe I missed something?

@jedgar1mx
Copy link
Author

I'm using master acquia/blt-travis@00bf8d7

I tried 1.0.3 but it would error out as soon as I tried to update locally.

Thanks @mikemadison13

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants