Skip to content

Upgrade PHP 8.5#2258

Merged
stakovicz merged 17 commits into
afup:masterfrom
stakovicz:feat-upgrade-php-8.5
Jun 8, 2026
Merged

Upgrade PHP 8.5#2258
stakovicz merged 17 commits into
afup:masterfrom
stakovicz:feat-upgrade-php-8.5

Conversation

@stakovicz
Copy link
Copy Markdown
Contributor

On met à jour PHP 8.5.
J'ai essayé de faire des commits atomic, mais c'est pas évident.

J'ai passé un coup de Rector et j'ai du adapter quelques tests à cause de la montée de version de smalot/pdfparser

@stakovicz stakovicz self-assigned this May 27, 2026
@stakovicz stakovicz requested review from Mopolo and agallou as code owners May 27, 2026 18:44
Copy link
Copy Markdown
Contributor

@Mopolo Mopolo left a comment

Choose a reason for hiding this comment

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

Un peu long à lire mais pas si compliqué en vrai. La majorité des modifs sont simples.

Comment thread tests/behat/bootstrap/EmailContext.php
Comment thread tests/behat/bootstrap/PdfContext.php
Comment thread tests/behat/bootstrap/PdfContext.php Outdated
Comment thread tests/behat/features/Admin/Events/FacturesEvennement.feature Outdated
Comment thread tests/behat/features/Admin/Tresorerie/DevisFactures.feature
@stakovicz stakovicz force-pushed the feat-upgrade-php-8.5 branch 2 times, most recently from 9e2a4f5 to 7e52eb9 Compare June 1, 2026 19:56
Comment thread app/config/reference.php
Copy link
Copy Markdown
Contributor

@Mopolo Mopolo left a comment

Choose a reason for hiding this comment

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

🚀

@stakovicz stakovicz force-pushed the feat-upgrade-php-8.5 branch from 9c62698 to b6500d1 Compare June 5, 2026 19:39
@stakovicz stakovicz force-pushed the feat-upgrade-php-8.5 branch from b6500d1 to 65e6961 Compare June 5, 2026 19:56
@stakovicz stakovicz force-pushed the feat-upgrade-php-8.5 branch from e747da3 to 2b4ac46 Compare June 6, 2026 11:37
@stakovicz stakovicz merged commit f9363a6 into afup:master Jun 8, 2026
13 of 14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants