Skip to content
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

Drop php 7.3 and add missing typehints #1463

Merged

Conversation

jordisala1991
Copy link
Member

@jordisala1991 jordisala1991 commented Dec 21, 2021

Subject

I am targeting this branch, because this breaks BC.

Closes #1438

Changelog

### Removed
- Removed support for PHP 7.3

@SonataCI
Copy link
Collaborator

Could you please rebase your PR and fix merge conflicts?

@jordisala1991 jordisala1991 force-pushed the improvement/drop-php-7.3 branch from 479410a to d4a91b7 Compare December 27, 2021 13:02
@jordisala1991 jordisala1991 changed the title [WIP] Drop php 7.3 and add missing typehints Drop php 7.3 and add missing typehints Dec 27, 2021
@jordisala1991 jordisala1991 force-pushed the improvement/drop-php-7.3 branch from d4a91b7 to 6f7de68 Compare December 27, 2021 13:03
@SonataCI
Copy link
Collaborator

Could you please rebase your PR and fix merge conflicts?

@jordisala1991 jordisala1991 force-pushed the improvement/drop-php-7.3 branch from 6f7de68 to 009a2fb Compare December 27, 2021 13:05
@jordisala1991 jordisala1991 marked this pull request as ready for review December 27, 2021 13:05
@jordisala1991 jordisala1991 force-pushed the improvement/drop-php-7.3 branch from 4824b9d to 4f2c810 Compare December 27, 2021 13:29
@jordisala1991
Copy link
Member Author

Dev-kit PR: sonata-project/dev-kit#1830

VincentLanglet
VincentLanglet previously approved these changes Dec 27, 2021
@jordisala1991 jordisala1991 merged commit 5a28675 into sonata-project:5.x Dec 27, 2021
@jordisala1991 jordisala1991 deleted the improvement/drop-php-7.3 branch December 27, 2021 14:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants