Fixed misplaced NoDiscard attribute

[ondrejmirtes]

It was above method signature for PHP 5.5 to 8.2.

[LolGleb]

Thanks for the PR! I understand adding #[NoDiscard] to the 8.3+ version makes sense. However, I'm curious why you removed it from the 5.5-8.2 version? Both versions have the same immutable behavior, so shouldn't both warn that the return value should be captured?

[ondrejmirtes]

I removed it from The 5.5-8.2 version because #[NoDiscard] is a PHP 8.5+ feature so it didn't seem relevant?

Can you confirm that PhpStorm would take advantage of the attribute even when the language level is set to PHP 8.2?

[LolGleb]

You are absolutely right! The attribute does not affect earlier versions of PHP, so even if we leave the attribute for earlier versions, it simply won't do anything