Proper handling of union types when used as a typehints for route parameters

[romalytvynenko]

fixes #219
fixes #385

[cwray-tech]

@romalytvynenko are these changes a part of a release? I'm still getting an error due to union types.

[romalytvynenko]

@cwray-tech can you confirm you still have the issue on 0.10.12? If so, please share the current code that fails. Thanks!

[cwray-tech]

@romalytvynenko No I was on v0.8.6.. Unfortunately, can't upgrade either due to a composer conflict. pestphp/pest-plugin-drift requires nikic/php-parser ^4.17.1 while this package requires greater than 5.

[romalytvynenko]

@cwray-tech Yeah, 0.9.0 and later versions have this breaking change. Which is the version of Laravel you're using?

[cwray-tech]

We're using laravel 10 unfortunately.

[cwray-tech]

Do you know of any work around to fix the union types issue on our end?

[romalytvynenko]

@cwray-tech Laravel 10 is supported though!

I can see that pest drift required php-parser v5 as well: https://github.com/pestphp/pest-plugin-drift/blob/2.x/composer.json

Any chance you can try updating both dependencies?

[cwray-tech]

@romalytvynenko I removed the package and it worked. But.. running into other issues related to our app.

[romalytvynenko]

@cwray-tech let me know if you need any help!