Skip to content

chore(release): bump php ci alpine version#8581

Merged
keradus merged 1 commit intoPHP-CS-Fixer:masterfrom
gnought:chore/bump-ci-php-alpine-version
Apr 18, 2025
Merged

chore(release): bump php ci alpine version#8581
keradus merged 1 commit intoPHP-CS-Fixer:masterfrom
gnought:chore/bump-ci-php-alpine-version

Conversation

@gnought
Copy link
Copy Markdown
Contributor

@gnought gnought commented Apr 14, 2025

Bump PHP-CS-Fixer docker alpine version

@coveralls
Copy link
Copy Markdown

Coverage Status

coverage: 94.872%. remained the same
when pulling 314dfcc on gnought:chore/bump-ci-php-alpine-version
into eb32dab on PHP-CS-Fixer:master.

alpine-version: '3.21'
- php-version: '8.3'
alpine-version: '3.18'
alpine-version: '3.21'
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

any idea how to avoid declaring the list in 2 places?
as we have 2 places, they diverge over time ;(

https://github.com/PHP-CS-Fixer/PHP-CS-Fixer/blob/master/compose.yaml#L28

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Yes it could be avoidable, but will involve quite big changes or refactoring on Dockerfile, compose files and some workflows. I could work on a new feature branch as a PoC.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@keradus keradus merged commit 05186fa into PHP-CS-Fixer:master Apr 18, 2025
30 checks passed
@gnought gnought deleted the chore/bump-ci-php-alpine-version branch April 18, 2025 15:06
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.

3 participants