[Step 2 Upgrade] Bump requirement to php 8.0

[samsonasik]

@lisachenko here the 2nd step 👍 , bump to php 8.0

[lisachenko]

@samsonasik There might be good parts which can be cherry-picked from the #122 or #121 - both of them were not merged.

[samsonasik]

I will check 👍

[samsonasik]

The current skipped testGetModifiers() seems needs separate PR:

https://github.com/samsonasik/parser-reflection/blob/f220c210ca2fc443b71466c864325d9ad0bc0470/tests/ReflectionClassTest.php#L31-L35

@lisachenko Ready to merge 👍

[lisachenko]

The current skipped testGetModifiers() seems needs separate PR:

https://github.com/samsonasik/parser-reflection/blob/f220c210ca2fc443b71466c864325d9ad0bc0470/tests/ReflectionClassTest.php#L31-L35

@lisachenko Ready to merge 👍

Awesome work! 👍 Let me check this today in the evening )

[lisachenko]

Question here: do you plant to add PHP8 Attributes as separate independent PR?

[samsonasik]

If you mean ReturnTypeWillChange attribute, that seems on php 8.1, the PR is step by step which this target is to make phpunit green first.

Otherwise, I may need failing test case for that to be implemented.

[scrutinizer-notifier]

The inspection completed: No new issues

[lisachenko]

Approved, thanks