Hey!

Thanks for your PR. You are targeting branch "7.4" but it seems your PR description refers to branch "7.4 for features / 6.4, 7.2, or 7.3 for bug fixes".
Could you update the PR description or change target branch? This helps core maintainers a lot.

Cheers!

Carsonbot

I wrote my opinion on#60429 (comment) before:

Ivoted against #[NoDiscard], so did@fabpot and I'm going to be 👎 here also: this attribute only contributes to make reading PHP code look like syntax soup, with no real value.

I see, never thought about have them on wither methods, as I maybe not personally fan of that pattern 😆.

For me it really only thought it handy forSecurity related things, likeUriSigner::checkRequest,Security::isGranted, ... where dangerous things could happen if somebody forget to check the false case (which we did have in one of our projects, where people thought checkRequest would throw exception and not require handle the return).