Skip to content
This repository has been archived by the owner on Jul 28, 2022. It is now read-only.

Allow Support PHP 8.0 #545

Merged
merged 1 commit into from Feb 24, 2021
Merged

Allow Support PHP 8.0 #545

merged 1 commit into from Feb 24, 2021

Conversation

Yozhef
Copy link
Contributor

@Yozhef Yozhef commented Jan 27, 2021

Changelog

### Added
- Add support for PHP 8.0

@VincentLanglet
Copy link
Member

VincentLanglet commented Jan 27, 2021

A release is needed for sonata-project/datagrid-bundle

I'm on it: sonata-project/SonataDatagridBundle#317

@phansys phansys mentioned this pull request Jan 27, 2021
1 task
@VincentLanglet
Copy link
Member

Blocker by sonata-project/doctrine-orm-admin-bundle now

@jordisala1991
Copy link
Member

We need the dev kit PR @Yozhef

@Yozhef
Copy link
Contributor Author

Yozhef commented Feb 24, 2021

@jordisala1991 yes, 5 min =)

@Yozhef
Copy link
Contributor Author

Yozhef commented Feb 24, 2021

@jordisala1991 sonata-project/dev-kit#1429

@VincentLanglet VincentLanglet merged commit 72ecf27 into sonata-project:3.x Feb 24, 2021
@wbloszyk
Copy link
Member

wbloszyk commented Mar 1, 2021

@sonata-project/contributors @Yozhef
We have problem here. Some dependencies are not allowed for php 8.0.

composer why-not php 8.0
php-amqplib/php-amqplib  2.12.2  requires  php (>=5.6.3,<8.0)  

@VincentLanglet
Copy link
Member

@Yozhef
Copy link
Contributor Author

Yozhef commented Mar 1, 2021

@wbloszyk php-amqplib/php-amqplib in master support - ned wait release =(

@wbloszyk
Copy link
Member

wbloszyk commented Mar 1, 2021

@wbloszyk php-amqplib/php-amqplib in master support - ned wait release =(

My bad, "php": ">=7.0",.

@Yozhef
Copy link
Contributor Author

Yozhef commented Mar 1, 2021

@wbloszyk ви праві зараз остання версія https://packagist.org/packages/php-amqplib/php-amqplib#2.12.2 не підтримує PHP 8.0 = (php:> = 5.6.3, <8.0 =( But its requered dev - deps I can't say for sure if it's important to abandon version 8 for now

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants