Emanuele Panzeri

Results 130 comments of Emanuele Panzeri

> @thePanz what issues do you see? Not sure, I did not mention any issue.. I might not get your question, sorry. Yes, I noticed that this PR (and the...

can phpstan identify those unused private properties?

the baseline is quite big, we should use the php format of the baseline to improve performances in the analysis hint: just generate the baseline with .php as exension, phpstan...

We should use `phpstan.dist.neon` as IDEs tend to recognize the type by the extension (this still works for phpstan)

> > the baseline is quite big, we should use the php format of the baseline to improve performances in the analysis > > hint: just generate the baseline with...

> I missed that point of your argue. No problem, i had a performance issue on another project for that reason, thus I would always opt for the php version...

php 8.4 is planned for December 2024 :sweat_smile: I had to check! :) It might be too early to have it now? Would be cool to mark php 8.4 pipeline...

I really like the approach from @tacman where twig-components can be used instead of macros. Watching [Ryan Weaver](https://www.youtube.com/@weaverryan) implementing that, he had an idea about having tabler icons being twig...

> I did this PR, but now I'm thinking that it's actually the code base doctrine, so it would be closer to the doctrine-current project if we use the doctrine...

> @bepsvpt I appreciate the PR, but we are unfortunately unable to entertain a full rewrite of the library, because we want to maintain consistency in the code layout between...