For context it was added by @smnandre in symfony/ux
symfony/ux#1166
symfony/ux#1793

For context it was added by @smnandre in symfony/ux

... only on the website (for now) :)

Regarding the suggestion, did you run it on the Symfony codebase ? Does it change a lot of files ? Only in "test / fixtures" or in Resources also ?

That'd be nice to know the "impact" i think :)

Regarding the suggestion, did you run it on the Symfony codebase ? Does it change a lot of files ? Only in "test / fixtures" or in Resources also ?

That'd be nice to know the "impact" i think :)

I didn't run it everywhere but I already found some issues like #54051

May I propose to add the tool twig-cs-fixer (repo) in this repo as a dev dependency

None of the tools we use (PHPUnit, PHP-CS-Fixer, Psalm) is an actual dev dependency on this repo. I don't think we want to change that.

actual dev dependency on this repo

yes I perhaps miswrite my purpose, more like adding a twig-cs-fix config file in the repo as its done for the php-cs-fixer
and from time to time, run it or via CI

did you run it on the Symfony codebase ?

nope, but I can of course and report back here when I 've done it

with a default config file, and the latest version of the fixer
on symfony/symfony 7.1, only src/Bundles
94noni@5f57bab

looks like major things are spacing inside arrays and around operators:

- {{ include('@WebProfiler/Profiler/toolbar_item.html.twig', { link: profiler_url }) }}
+ {{ include('@WebProfiler/Profiler/toolbar_item.html.twig', {link: profiler_url}) }}

and

- {{ profiler_dump(usage.trace, maxDepth=2) }}
+ {{ profiler_dump(usage.trace, maxDepth = 2) }}

wdyt?

Should it be fabbot's responsibility?

@chalasr If so, it should probably be Symfony specific then (at least at first) ?

@smnandre Perhaps yes, I don't know much about fabbot internals as it's not public code yet. I'd expect it to run tools like php-cs-fixer only if a matching config file exists in the project. I may be wrong though.
What I'm pretty sure is, if we check twig files as part of the CI, only the files that changed in the given PR/commit should be checked (which is how fabbot operates).

lets close for now, perhaps will be relevant when/if twig/twig comes inside this repo as symfony/twig later on
thx anyway for commenting here

Friendly ping to commentors :)

With recent twig all new features etc, i really think it could be relevant to add such tool in main repo

Wdyt of restarting this ?
Previous diff run on 7.1 94noni@5f57bab