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

Make docker-compose bin configurable #760

Merged
merged 2 commits into from
Mar 24, 2023

Conversation

andytson-inviqa
Copy link
Contributor

@andytson-inviqa andytson-inviqa commented Mar 24, 2023

Allow it to be configurable for switching to docker compose v2 as exec-plugin later

also fix an issue where some docker compose v2 versions delete all containers on docker compose rm with no arguments

@andytson-inviqa andytson-inviqa added enhancement New feature or request harness-magento1 Magento 1.x harness harness-magento2 Magento 2.x harness harness-php PHP harness harness-wordpress Wordpress harness harness-symfony Symfony harness harness-drupal Drupal harness, since 1.5.x labels Mar 24, 2023
@andytson-inviqa andytson-inviqa force-pushed the feature/docker-compose-bin-config branch from b8d7afa to 5bfdfab Compare March 24, 2023 15:03
kierenevans
kierenevans previously approved these changes Mar 24, 2023
@andytson-inviqa andytson-inviqa merged commit 0b47b04 into 1.5.x Mar 24, 2023
@andytson-inviqa andytson-inviqa deleted the feature/docker-compose-bin-config branch March 24, 2023 17:28
@andytson-inviqa andytson-inviqa added harness-all Changes required to all harness directories and removed harness-magento1 Magento 1.x harness harness-magento2 Magento 2.x harness harness-php PHP harness harness-wordpress Wordpress harness harness-symfony Symfony harness harness-drupal Drupal harness, since 1.5.x labels Mar 24, 2023
@andytson-inviqa andytson-inviqa added this to the 1.5.0 milestone May 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request harness-all Changes required to all harness directories
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants