minor: Let PhpdocLineSpan fixer detect docblocks when seperator from token with attribute

[bobvandevijver]

This fix allows the phpdoc_line_span fixer to detect attributes between a docblock and the token it is working on.

It now updates (depending on the configuration)

/**
 * @return void
 */
#[Attribute1, Attribute2]
public function hello() {}

to

/** @return void */
#[Attribute1, Attribute2]
public function hello() {}

while it did not touch this structure before.

[SpacePossum]

Thanks for the PR! and sorry for the long wait.

I left a little minor, but with or without it applied I think this PR RTM

[SpacePossum]

thanks for the changes @bobvandevijver 👍

@julienfalque can you allow the workflow to run? thanks!

[coveralls]

Coverage increased (+0.002%) to 92.905% when pulling e73fb9a on bobvandevijver:phpdoc_line_span_attributes into 40716fc on FriendsOfPHP:master.

[bobvandevijver]

Maybe a rebase would help to fix the static analysis?

[julienfalque]

It should be fixed by #6506.

[bobvandevijver]

Looks like phpstan is finding 13 errors, but they are all in other files...

[SpacePossum]

yeah I think what it really wants is some type docs on the newly created dataprovider, but not 100% sure, maybe Julien has an idea on this one :)

thanks for change from \Generator to iterable 👍

[julienfalque]

Yep, I think the reported error is the last that matches the ignore regexp, not necessarily the one that is added by your changes.

[bobvandevijver]

@julienfalque I actually had mixed first, but changed it back to string as that was more commonly used in the other tests 😅

[julienfalque]

I think mixed is best suited because we can't be sure options will always be string. Anyway, not very important since data providers are not checked against test parameters.

[julienfalque]

Thank you @bobvandevijver.

[SpacePossum]

thanks @bobvandevijver , nice one :)