Skip to content

Conversation

@ghost
Copy link

@ghost ghost commented Oct 14, 2020

i just run robo build:docs on this.

@ThomasLandauer
Copy link
Owner

Um, sorry, this got already merged, see Codeception#28
So what am I supposed to do with your PR now?

BTW: I didn't get a notification from GitHub about this yesterday :-(

@ghost
Copy link
Author

ghost commented Oct 15, 2020

@ThomasLandauer don't worry, I'll present this same PR based on the master branch.

@ThomasLandauer
Copy link
Owner

OK, thanks, I'm deleting the ThomasLandauer:patch-2 branch

@ghost ghost deleted the ThomasLandauer branch October 15, 2020 14:22
ThomasLandauer added a commit that referenced this pull request Sep 8, 2022
The line `use JsonSerializable;` of this file is not being passed on to `tests/Support/_generated/AcceptanceTesterActions.php`. And with the current phpDoc
```php
@param array|string|JsonSerializable $params
```
... phpstan is correctly reporting:
> Parameter #2 $params of method Tests\Support\AcceptanceTester::sendPatch() expects array|string|Tests\Support\_generated\JsonSerializable ...

So the question is: Is there a way to include the needed `use`s in the generated `TesterAction`s` Or should all occurrences of such "root" classes in the code get prefixed with a `\`?
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants