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

Update our Doocker setup to use PHP 8.1 images #392

Merged

Conversation

tbsiqueira
Copy link
Contributor

@tbsiqueira tbsiqueira commented Jun 23, 2023

Since we are not supporting PHP 8.1, let's update our docker images to use PHP 8.1

https://getopensocial.atlassian.net/browse/PROD-25687

🚀 🔥 Related PRs 🔥 🚀

This is a PR from a series of 5 PRs that should be merged in the following order:

  1. Issue #3373075 by tbsiqueira: Update dev/test dependencies open_social_dev#70
  2. Issue #3373172 by tbsiqueira: Update our tugboat docker image with one supporting PHP 8.1 open_social#3460
  3. Issue #3373075 by tbsiqueira: Update from drupal/core-recommended to drupal/core open_social#3457
  4. Update our Doocker setup to use PHP 8.1 images #392 ( this is just a private repository that we use for development, this will not be available for public users )
  5. Issue #3368689 by tbsiqueira: Update behat tests to use the new compatible php 8.1 images open_social#3439

They are ALL BC and should not cause any breaking change for our local development

Revert change on composer.json to require dev-main, and update docker-compose files
to use the latest released open_social_docker images featuring PHP 8.1 compatibility
@tbsiqueira tbsiqueira changed the title Update composer and docker-compose and docker-compose-ci files Update our Behat setup to use PHP 8.1 images Jul 7, 2023
@tbsiqueira tbsiqueira changed the title Update our Behat setup to use PHP 8.1 images Update our Doocker setup to use PHP 8.1 images Jul 7, 2023
@tbsiqueira tbsiqueira merged commit e3b926a into master Jul 10, 2023
1 check failed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant