Skip to content

use phpunit attributes#562

Merged
nikophil merged 1 commit intozenstruck:1.xfrom
jrushlow:feature/phpunit-attributes
Mar 20, 2024
Merged

use phpunit attributes#562
nikophil merged 1 commit intozenstruck:1.xfrom
jrushlow:feature/phpunit-attributes

Conversation

@jrushlow
Copy link
Contributor

Starting in PHPUnit 11 - the use of metadata in doc-comments is hard deprecated. Results in tests passing as expected but PHPUnit deprecation warnings everywhere...

@jrushlow
Copy link
Contributor Author

Not sure if ya'll use any sort of @legacy comments like we do in MakerBundle. If so, let me know and I'll add them to the doc-comments. E.g. @after - @legacy - can be removed when PHPUnit <10 support is dropped.

@nikophil
Copy link
Member

nikophil commented Mar 20, 2024

Hi @jrushlow

sorry to have been quite on this lately.

Because we only use phpunit ^9.5, I don't think we'll need any @group legacy for now?

could rebase your PR, please?

@jrushlow jrushlow force-pushed the feature/phpunit-attributes branch from e5ab9f5 to 79a6f97 Compare March 20, 2024 07:57
@jrushlow
Copy link
Contributor Author

@nikophil rebase done

don't think the ci failure is related.. looks like a php-cs-fixer type of problem

@nikophil
Copy link
Member

absolutely, I'll fix this today

@nikophil
Copy link
Member

thanks @jrushlow

@nikophil nikophil merged commit 68148e1 into zenstruck:1.x Mar 20, 2024
@jrushlow jrushlow deleted the feature/phpunit-attributes branch March 20, 2024 15:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants