Skip to content

Conversation

@TimWolla
Copy link
Contributor

@TimWolla TimWolla commented Jan 1, 2026

Copy link
Member

@tianon tianon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great, thank you!

There's a tiny bit more we can clean up in versions.sh, but I can do that post-merge:

php/versions.sh

Lines 94 to 101 in 6854964

if [[ "$rcVersion" = '8.1' ]] && [[ "$suite" = 'alpine3.23' ]]; then
# Keep PHP 8.1 with Alpine 3.21 default until end of year; see also https://github.com/docker-library/php/blob/9ab2e4b37addffaa10f06d9e5f54f7bd1f5ef18f/generate-stackbrew-library.sh#L120
continue
fi
if [[ "$rcVersion" != '8.1' ]] && [[ "$suite" = 'alpine3.21' ]]; then
# Keep Alpine 3.21 just for 8.1
continue
fi
❤️

@tianon tianon merged commit a56a1b3 into docker-library:master Jan 2, 2026
58 checks passed
tianon added a commit that referenced this pull request Jan 2, 2026
docker-library-bot added a commit to docker-library-bot/official-images that referenced this pull request Jan 2, 2026
Changes:

- docker-library/php@6c91a369: Remove more 8.1 special cases
- docker-library/php@a56a1b39: Merge pull request docker-library/php#1639 from TimWolla/remove-eol-8.1
- docker-library/php@68549643: Update 8.3-rc to 8.3.30RC1
- docker-library/php@0d2677ff: Update 8.5-rc to 8.5.2RC1
- docker-library/php@be2cb18a: Update 8.4-rc to 8.4.17RC1
- docker-library/php@9cf5f245: Remove EOL 8.1
tianon added a commit to docker-library/official-images that referenced this pull request Jan 2, 2026
Changes:

- docker-library/php@6c91a369: Remove more 8.1 special cases
- docker-library/php@a56a1b39: Merge pull request docker-library/php#1639 from TimWolla/remove-eol-8.1
- docker-library/php@68549643: Update 8.3-rc to 8.3.30RC1
- docker-library/php@0d2677ff: Update 8.5-rc to 8.5.2RC1
- docker-library/php@be2cb18a: Update 8.4-rc to 8.4.17RC1
- docker-library/php@9cf5f245: Remove EOL 8.1

Co-authored-by: Docker Library Bot <doi+docker-library-bot@docker.com>
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