Skip to content

[Testing] [PHPUnit-Bridge] Troubleshooting #14917

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
May 25, 2021

Conversation

sylfabre
Copy link
Contributor

@sylfabre sylfabre commented Feb 2, 2021

This PR documents a new solution to an issue I'm having with ClockMock related to https://bugs.php.net/bug.php?id=76788.

I was the original author of the current Troubleshooting section and came across another use-case

@sylfabre sylfabre force-pushed the phpunit_bridge branch 2 times, most recently from ca960e1 to b50238c Compare February 2, 2021 14:06
@sylfabre
Copy link
Contributor Author

sylfabre commented Feb 3, 2021

I don't understand what's wrong with https://github.com/symfony/symfony-docs/runs/1815088336?check_suite_focus=true

It looks like the syntax I can find in other files like https://github.com/symfony/symfony-docs/blob/5.x/components/yaml.rst line 179

@carsonbot carsonbot changed the title [PHPUnit-Bridge] Troubleshooting [Testing] [PHPUnit-Bridge] Troubleshooting May 25, 2021
@javiereguiluz javiereguiluz changed the base branch from 5.4 to 4.4 May 25, 2021 14:24
@javiereguiluz javiereguiluz requested a review from xabbuh as a code owner May 25, 2021 14:24
@javiereguiluz javiereguiluz merged commit 3dc74f6 into symfony:4.4 May 25, 2021
@javiereguiluz
Copy link
Member

Thanks Sylvain! The reported error was about using .. code-block:: php which is not allowed in Symfony Docs because we use instead the shortcut ::

We fixed that while merging this. Thanks!

@sylfabre
Copy link
Contributor Author

@javiereguiluz thank you for the merge and the explanation 👌

Do I need to submit a PR for 5.x too? Or does it gets merged "there" too?

@javiereguiluz
Copy link
Member

There's no need to submit PR for upper branches because we "upmerge" all PR in all supported branches. Cheers!

@sylfabre sylfabre deleted the phpunit_bridge branch May 25, 2021 21:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants