Add phpstan-return annotation to translateText

[VincentLanglet]

Hi @daniel-jones-deepl,

When using static analysis tool like PHPStan or Psalm, it is possible to explain that the return type of translateText

• is a TextResult when the input is a string
• is an array when the input is an array

Cf https://phpstan.org/blog/phpstan-1-6-0-with-conditional-return-types#conditional-return-types

This would be helpful to help the tool understanding better the result type and avoid useless extra checks.

[JanEbbing]

Sure, we have this in python already. I'll review this today.

[JanEbbing]

LGTM

[VincentLanglet]

Thanks. Do you think a new tag would be possible ? Or is one planned soon ?

[JanEbbing]

Yes, I will make a new release, I had some issue with it on friday

[JanEbbing]

Hi @VincentLanglet , sorry I just noticed a new release would include release for Arabic in the new Version. Is this urgent or can it wait a few days?

[VincentLanglet]

Hi @VincentLanglet , sorry I just noticed a new release would include release for Arabic in the new Version. Is this urgent or can it wait a few days?

Hi @JanEbbing , it's not urgent.

Also maybe the new release could include a fix for #43 (comment) if the issue is on this library side.

[JanEbbing]

I think that issue is API-side. I remember a case where a Dotnet user had the same problem for a different text - ill report it to the responsible team, but not sure this can be fixed quickly.

[JanEbbing]

@VincentLanglet released in v1.7.0 now