Skip to content

[Clock] Add attributes to support PHPUnit 10 + 11 #54129

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
Mar 1, 2024

Conversation

ruudk
Copy link
Contributor

@ruudk ruudk commented Mar 1, 2024

Q A
Branch? 6.4
Bug fix? yes
New feature? no
Deprecations? no
Issues Fix #...
License MIT

While upgrading to PHPUnit 11 I noticed the following deprecation:

Metadata in doc-comments is deprecated and will no longer be supported in PHPUnit 12. Update your test code to use attributes instead.

If we add the attribute too, it stops complaining. So this most be a way to support PHPUnit 11 and older versions.

@carsonbot
Copy link

Hey!

Thanks for your PR. You are targeting branch "7.1" but it seems your PR description refers to branch "6.4".
Could you update the PR description or change target branch? This helps core maintainers a lot.

Cheers!

Carsonbot

@ruudk ruudk changed the base branch from 7.1 to 6.4 March 1, 2024 14:01
PHPUnit 11 will complain with the following deprecation:
```
Metadata in doc-comments is deprecated and will no longer be supported in PHPUnit 12.
Update your test code to use attributes instead.
```

If we add the attribute too, it stops complaining. So this most be a way to support PHPUnit 11 and
older versions.
@ruudk ruudk force-pushed the phpunit-11-clock branch from 194d3c4 to 0478d54 Compare March 1, 2024 14:02
@ruudk
Copy link
Contributor Author

ruudk commented Mar 1, 2024

@carsonbot I updated the base branch.

@ruudk
Copy link
Contributor Author

ruudk commented Mar 1, 2024

@stof Seems that Carson is stuck, can you update it to 6.4 milestone?

@stof stof modified the milestones: 7.1, 6.4 Mar 1, 2024
@nicolas-grekas
Copy link
Member

Thank you @ruudk.

@nicolas-grekas nicolas-grekas merged commit ff08e9e into symfony:6.4 Mar 1, 2024
@ruudk ruudk deleted the phpunit-11-clock branch March 1, 2024 22:42
This was referenced Mar 4, 2024
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.

5 participants