Skip to content

[ci] use annotations when attributes are not supported #1064

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
Feb 10, 2022

Conversation

jrushlow
Copy link
Collaborator

@jrushlow jrushlow commented Feb 8, 2022

DoctrineBundle >=2.4's flex recipe uses attributes by default - regardless if you're using PHP < 8.0. We need to use attributes in our tests by default unless doctrine and/or php does not support them.

@jrushlow jrushlow force-pushed the fix/ci-attribute-failures branch from 4d1c29e to 1812c0a Compare February 8, 2022 15:30
@jrushlow jrushlow changed the title WIP - [ci] use annotations when attributes are not supported [ci] use annotations when attributes are not supported Feb 8, 2022
@jrushlow jrushlow marked this pull request as ready for review February 8, 2022 15:37
@jrushlow
Copy link
Collaborator Author

jrushlow commented Feb 8, 2022

CI test failures are related to panther tests in 4.4 and make registration tests.

@jrushlow jrushlow force-pushed the fix/ci-attribute-failures branch from 1812c0a to 04f8fea Compare February 10, 2022 14:21
@jrushlow jrushlow force-pushed the fix/ci-attribute-failures branch from f4b9628 to 00797b6 Compare February 10, 2022 14:41
@weaverryan
Copy link
Member

Thanks Jesse!

@weaverryan weaverryan merged commit 73ecf23 into symfony:main Feb 10, 2022
@jrushlow jrushlow deleted the fix/ci-attribute-failures branch February 10, 2022 14:59
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