Psalm annotations about automapper-plus HOT 3 OPEN

Hi @Jean85, I don't have any experience with psalm myself, unfortunately. I'm definitely all for a PR that makes life easier for psalm users, but could you explain the difference between replacing the annotations and adding them? Are there any (dis)advantages between the two?

From a quick glance at the docs my understanding is that the @psalm- annotations are for specific psalm functionality (e.g. type syntax or info not currently possible in regular phpbloc). If that's the case then I lean towards keeping (and/or improving) the PHPdocs and adding @psalm tags where needed.

from automapper-plus.

Yes, you got it right: @psalm-* annotations are more specific and can be put alongside the normal one, so to not break tools (like IDEs) that do not understand them.

I think that the best way to make it work correctly would be to introduce Psalm into your codebase and CI, so to have a tool that checks the correctness of such annotations; are you open to a PR of such kind?

from automapper-plus.

Yes absolutely! I've added PHPStan to the 2.0 branch, but that branch is still very much WIP. Psalm and PHPStan can perfectly coexist together, so feel free to open up a PR to the master branch. It'll be my problem to merge it in the 2.0 branch then.

from automapper-plus.