Skip to content

Comments

Add support for Twig CS Fixer code checks#368

Merged
Ocramius merged 1 commit intolaminas:1.33.xfrom
silarhi:claude/add-twig-cs-fixer-MPpWB
Jan 26, 2026
Merged

Add support for Twig CS Fixer code checks#368
Ocramius merged 1 commit intolaminas:1.33.xfrom
silarhi:claude/add-twig-cs-fixer-MPpWB

Conversation

@guillaume-sainthillier
Copy link

Q A
Documentation yes/no
Bugfix yes/no
BC Break yes/no
New Feature yes
RFC yes/no
QA yes/no

Description

Adds detection for Twig-CS-Fixer based on the presence of .twig-cs-fixer.php or .twig-cs-fixer.dist.php configuration files.

Copy link
Member

@gsteel gsteel left a comment

Choose a reason for hiding this comment

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

LGTM! Thank you @guillaume-sainthillier

I'll wait for another approving review before merging 👍

@gsteel gsteel added this to the 1.33.0 milestone Jan 26, 2026
@gsteel gsteel requested a review from a team January 26, 2026 14:18
@gsteel
Copy link
Member

gsteel commented Jan 26, 2026

@guillaume-sainthillier guillaume-sainthillier force-pushed the claude/add-twig-cs-fixer-MPpWB branch 2 times, most recently from 92eaf86 to adf1941 Compare January 26, 2026 14:31
Adds detection for Twig-CS-Fixer based on the presence of
`.twig-cs-fixer.php` or `.twig-cs-fixer.dist.php` configuration files.

Signed-off-by: Guillaume Sainthillier <guillaume.sainthillier@gmail.com>
@guillaume-sainthillier guillaume-sainthillier force-pushed the claude/add-twig-cs-fixer-MPpWB branch from adf1941 to 30d6045 Compare January 26, 2026 14:32
@guillaume-sainthillier
Copy link
Author

@gsteel done :)

@Ocramius Ocramius self-assigned this Jan 26, 2026
@Ocramius
Copy link
Member

Thanks @guillaume-sainthillier!

@Ocramius Ocramius merged commit 6383012 into laminas:1.33.x Jan 26, 2026
51 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.

4 participants