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

Chore/update docker network #170

Merged
merged 6 commits into from
Feb 12, 2025
Merged

Chore/update docker network #170

merged 6 commits into from
Feb 12, 2025

Conversation

snim2
Copy link
Contributor

@snim2 snim2 commented Feb 7, 2025

  • Remove Beanstalk
  • Remove version directive from docker-compose.yml
  • Use PHP8 on Docker
  • Shell scripts pass shellcheck
  • CI pipeline deprecates PHP 7.4

Design notes

Note that I haven't made much effort to update the theme here because we don't really use it any more. In the longer term I'm wondering if we should swap that out for the govuk-theme.

Steps for testing

  • Check CI pipeline
  • Run ./script/setup
  • Run ./script/server

@snim2 snim2 force-pushed the chore/update-docker-network branch from e729bf3 to e033078 Compare February 7, 2025 13:26
snim2 added 6 commits February 7, 2025 13:40
We don't deploy Beanstalk anywhere now, and very
few sites run the replacement service (SQS), so
it makes sense just to remove this.
We currently deploy new sites with PHP 8.3 and
develop with 8.2, so for now it makes sense to
have 8.2 as the default version of PHP in our
template.
@snim2 snim2 force-pushed the chore/update-docker-network branch from e033078 to 99649f0 Compare February 7, 2025 13:41
@snim2 snim2 marked this pull request as ready for review February 7, 2025 13:50
@snim2 snim2 merged commit 1825aef into main Feb 12, 2025
8 checks passed
@snim2 snim2 deleted the chore/update-docker-network branch February 12, 2025 09:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants