…es manager (Kocal)

This PR was merged into the 2.x branch.

Discussion
----------

[CI] Rework `app-tests` job to use `npm` as user's packages manager

| Q             | A
| ------------- | ---
| Bug fix?      | no
| New feature?  | no <!-- please update src/**/CHANGELOG.md files -->
| Docs?         | no <!-- required for new features -->
| Issues        | Fix #... <!-- prefix each issue number with "Fix #", no need to create an issue if none exist, explain below instead -->
| License       | MIT

Following #2952, this way we ensure our packages will always be installable with `npm` (related to #2951)

Commits
-------

a9ea07d [CI] Rework app-tests job to use npm as user's packages manager

This PR was merged into the 2.x branch.

Discussion
----------

[CI] Fix npm releases with pnpm

| Q             | A
| ------------- | ---
| Bug fix?      | no
| New feature?  | no <!-- please update src/**/CHANGELOG.md files -->
| Docs?         | no <!-- required for new features -->
| Issues        | Fix #... <!-- prefix each issue number with "Fix #", no need to create an issue if none exist, explain below instead -->
| License       | MIT

Given https://docs.npmjs.com/using-private-packages-in-a-ci-cd-workflow, https://pnpm.io/using-changesets#add-a-publish-script, or pnpm/pnpm#3141

It should works better now.

⚠️ no need to release a 2.28.2 right now, I already manually published packages on npm for 2.28.1 version

Commits
-------

4061a42 [CI] Fix npm releases with pnpm

This PR was squashed before being merged into the 2.x branch.

Discussion
----------

[CI] Add Fabbot as a GitHub Actions

| Q             | A
| ------------- | ---
| Bug fix?      | no
| New feature?  | no <!-- please update src/**/CHANGELOG.md files -->
| Docs?         | no <!-- required for new features -->
| Issues        | Fix #... <!-- prefix each issue number with "Fix #", no need to create an issue if none exist, explain below instead -->
| License       | MIT

<!--
Replace this notice by a description of your feature/bugfix.
This will help reviewers and should be a good start for the documentation.

Additionally (see https://symfony.com/releases):
 - Always add tests and ensure they pass.
 - For new features, provide some code snippets to help understand usage.
 - Features and deprecations must be submitted against branch main.
 - Update/add documentation as required (we can help!)
 - Changelog entry should follow https://symfony.com/doc/current/contributing/code/conventions.html#writing-a-changelog-entry
 - Never break backward compatibility (see https://symfony.com/bc).
-->

refs:
- symfony/symfony#61260
- twigphp/Twig#4671
- https://github.com/symfony-tools/fabbot

Since the code formatting process if more transparent, maybe we will remove the job `coding-style-php` that manually execute PHP-CS-Fixer (we had many formatting issues in the past)

Commits
-------

23aa491 [CI] Add Fabbot as a GitHub Actions

This PR was merged into the 2.x branch.

Discussion
----------

Apply fabbot rules

| Q             | A
| ------------- | ---
| Bug fix?      | no
| New feature?  | no
| Docs?         | no
| Issues        | -
| License       | MIT

- no-void to test cases
- typos
- tweaks to exception messages

Commits
-------

7b135d0 Apply fabbot rules

…ons (roverwolf)

This PR was squashed before being merged into the 2.x branch.

Discussion
----------

[ux-icons] Fix value of xmlns attribute for imported svg icons

| Q             | A
| ------------- | ---
| Bug fix?      | yes
| New feature?  | no
| Docs?         | no
| Issues        | Fix #2958
| License       | MIT

Change the xmlns attribute to be `http://www.w3.org/2000/svg` for icons imported using `console ux:icons:import` so that the icons will correctly render in browsers and can be viewed by other image tools.

Commits
-------

bdf6270 [ux-icons] Fix value of xmlns attribute for imported svg icons

…ed` (smnandre)

This PR was squashed before being merged into the 2.x branch.

Discussion
----------

[LiveComponent] Fix LiveUrlSubscriber throw `MethodNotAllowed`

| Q             | A
| ------------- | ---
| Bug fix?      | yes
| New feature?  | no <!-- please update src/**/CHANGELOG.md files -->
| Docs?         | no <!-- required for new features -->
| Issues        | Fix #2960
| License       | MIT

> [!WARNING]
> Not sure this is enough, but If anyone wants to build on it, feel free - wont be around tomorrow.

---

The new LiveUrlSubscriber calls the Router to match the previous Url.

At that instant, the RoutingContext is most of the time in POST, and so calling router->match($previousUrl) would throw a MethodNotAllowedException

---

Most recent commit ([Catch MethodNotAllowedException](1da8a7b)) fixes _**this**_ issue ... but I think we have probably other use cases where it would not work.

My previous commit (reverted in the current state) [Force RequestsContext to "GET"](e81aad2) seems a more promising lead.

And the first one (reverted too in current state) [No LiveProps no change](1da8a7b)  was to avoid entirely the generation when no props were extracted from URL... sadly i don't think it's enough to say "no props should now be used in URL".. ?

---

Thank you `@zacharylund`  for the investigation and `@jmsche` for the perfect reproducer

Commits
-------

5b8d701 [LiveComponent] Fix LiveUrlSubscriber throw `MethodNotAllowed`