Skip to content

chore: Migrate Symfony XML configuration usage to the PHP one#2449

Merged
norberttech merged 1 commit into
flow-php:1.xfrom
stloyd:chore/sf-migrate-xml
Jun 14, 2026
Merged

chore: Migrate Symfony XML configuration usage to the PHP one#2449
norberttech merged 1 commit into
flow-php:1.xfrom
stloyd:chore/sf-migrate-xml

Conversation

@stloyd

@stloyd stloyd commented Jun 14, 2026

Copy link
Copy Markdown
Member

Symfony deprecated the XML configuration format in release 7.4, and removed it in 8.0. The recommended format is PHP one.

Should fix issues in tests like:

Flow\Bridge\Symfony\PostgreSqlBundle\Tests\Integration\Profiler\FlowPostgreSqlProfilerTest::test_panel_renders_executed_queries_in_the_profiler
Symfony\Component\Config\Exception\LoaderLoadException: Cannot load resource "@WebProfilerBundle/Resources/config/routing/profiler.xml". Make sure the "WebProfilerBundle" bundle is correctly registered and loaded in the application kernel class. If the bundle is registered, make sure the bundle path "@WebProfilerBundle/Resources/config/routing/profiler.xml" is not empty.

Change Log


Added

Fixed

Changed

  • Migrate Symfony XML configuration usage to the PHP one

Removed

Deprecated

Security

@stloyd stloyd requested a review from norberttech as a code owner June 14, 2026 10:46
@stloyd stloyd changed the title chore: Migrate Symfony XML usage to the PHP one chore: Migrate Symfony XML configuration usage to the PHP one Jun 14, 2026
@codecov

codecov Bot commented Jun 14, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.11%. Comparing base (e41f58b) to head (968904e).

Additional details and impacted files
@@             Coverage Diff              @@
##                1.x    #2449      +/-   ##
============================================
- Coverage     85.11%   85.11%   -0.01%     
- Complexity        0    21161   +21161     
============================================
  Files          1599     1599              
  Lines         65334    65334              
============================================
- Hits          55612    55611       -1     
- Misses         9722     9723       +1     
Components Coverage Δ
etl 88.32% <ø> (ø)
cli 89.40% <ø> (ø)
lib-array-dot 81.44% <ø> (ø)
lib-azure-sdk 64.44% <ø> (ø)
lib-doctrine-dbal-bulk 93.61% <ø> (ø)
lib-filesystem 85.03% <ø> (ø)
lib-types 90.06% <ø> (ø)
lib-parquet 70.10% <ø> (ø)
lib-parquet-viewer 82.26% <ø> (ø)
lib-snappy 89.38% <ø> (-0.45%) ⬇️
lib-dremel 0.00% <ø> (ø)
lib-postgresql 88.58% <ø> (ø)
lib-telemetry 85.95% <ø> (ø)
bridge-filesystem-async-aws 92.74% <ø> (ø)
bridge-filesystem-azure 90.45% <ø> (ø)
bridge-monolog-http 96.82% <ø> (ø)
bridge-monolog-telemetry 94.11% <ø> (ø)
bridge-openapi-specification 92.07% <ø> (ø)
symfony-http-foundation 78.57% <ø> (ø)
bridge-psr18-telemetry 100.00% <ø> (ø)
bridge-psr3-telemetry 97.84% <ø> (ø)
bridge-psr7-telemetry 100.00% <ø> (ø)
bridge-telemetry-otlp 89.89% <ø> (ø)
bridge-symfony-http-foundation-telemetry 89.47% <ø> (ø)
bridge-symfony-filesystem-bundle 90.20% <ø> (ø)
bridge-symfony-filesystem-cache 98.14% <ø> (ø)
bridge-symfony-postgresql-bundle 93.83% <ø> (ø)
bridge-symfony-postgresql-cache 94.41% <ø> (ø)
bridge-symfony-postgresql-messenger 98.80% <ø> (ø)
bridge-symfony-postgresql-session 93.65% <ø> (ø)
bridge-symfony-telemetry-bundle 80.80% <ø> (ø)
adapter-chartjs 84.05% <ø> (ø)
adapter-csv 91.16% <ø> (ø)
adapter-doctrine 90.79% <ø> (ø)
adapter-elasticsearch 92.07% <ø> (ø)
adapter-google-sheet 99.18% <ø> (ø)
adapter-http 72.34% <ø> (ø)
adapter-json 88.63% <ø> (ø)
adapter-logger 50.00% <ø> (ø)
adapter-parquet 77.70% <ø> (ø)
adapter-text 74.13% <ø> (ø)
adapter-xml 83.40% <ø> (ø)
adapter-avro 0.00% <ø> (ø)
adapter-excel 94.21% <ø> (ø)
adapter-postgresql 91.11% <ø> (ø)
bridge-phpunit-postgresql 75.30% <ø> (ø)
bridge-phpunit-telemetry 80.08% <ø> (ø)
bridge-phpstan-types 0.00% <ø> (ø)
bridge-postgresql-valinor 100.00% <ø> (ø)
🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@norberttech

Copy link
Copy Markdown
Member

Does it work also with 6.4? We are supporting right now 6.4 | 7.4 and 8.0+

@stloyd

stloyd commented Jun 14, 2026

Copy link
Copy Markdown
Member Author

Seems like the new type was added in 7.4, in 6.4 it's still XML one.

@stloyd stloyd marked this pull request as draft June 14, 2026 11:28
@stloyd stloyd force-pushed the chore/sf-migrate-xml branch 4 times, most recently from 1ae8671 to a4ebe82 Compare June 14, 2026 11:44
@stloyd stloyd force-pushed the chore/sf-migrate-xml branch from a4ebe82 to 968904e Compare June 14, 2026 11:51
@stloyd stloyd marked this pull request as ready for review June 14, 2026 11:54
@norberttech

Copy link
Copy Markdown
Member

hmm let me think about it, we might be able to drop 6.4 entirely

@norberttech norberttech merged commit 65357c2 into flow-php:1.x Jun 14, 2026
40 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants