Skip to content

Rename twig template folders and files to snake_case - #116

Open
loevgaard wants to merge 1 commit into
2.xfrom
snake-case-templates
Open

Rename twig template folders and files to snake_case#116
loevgaard wants to merge 1 commit into
2.xfrom
snake-case-templates

Conversation

@loevgaard

Copy link
Copy Markdown
Member

Fixes #115

The four templates move to snake_case paths per modern Symfony/Sylius convention (Form/themeform/theme, Admin/Order/Show/Payment/*admin/order/show/payment/*, with _operationsError also renamed to _operations_error), and all 8 reference sites follow (controller renders, the two prepended configs, and the four test expectations). CLAUDE.md's path reference updated too.

Free while 2.x is unreleased — 1.x shipped no templates, so no override paths exist in the wild and no upgrade note is needed, per the issue's BC analysis.

Verified: 73 tests, PHPStan (cold), ECS, lint:twig/lint:container green; the admin endpoint still responds after a cache clear. (Renames were done with two-step git mv so the case change is committed correctly despite the case-insensitive local filesystem.)

@codecov

codecov Bot commented Aug 10, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 79.60%. Comparing base (af2574a) to head (2beed0b).

Additional details and impacted files
@@            Coverage Diff            @@
##                2.x     #116   +/-   ##
=========================================
  Coverage     79.60%   79.60%           
  Complexity      114      114           
=========================================
  Files            16       16           
  Lines           510      510           
=========================================
  Hits            406      406           
  Misses          104      104           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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.

Rename twig template folders and files to snake_case

1 participant