Skip to content

build(deps): up PHP version to 8.2#1592

Open
MrKrasnov wants to merge 3 commits into
OpenVK:masterfrom
MrKrasnov:composer-php-up-version
Open

build(deps): up PHP version to 8.2#1592
MrKrasnov wants to merge 3 commits into
OpenVK:masterfrom
MrKrasnov:composer-php-up-version

Conversation

@MrKrasnov
Copy link
Copy Markdown

No description provided.

Copy link
Copy Markdown
Member

@WerySkok WerySkok left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Оказывается, у нас не так много мест, где упоминается PHP 7, но вот тут бы я добавил исправление:

if (!version_compare(PHP_VERSION, "7.3.0", ">=")) {

@MrKrasnov
Copy link
Copy Markdown
Author

@WerySkok а что конкретно ты видишь необходимым тут исправлять ? Вполне логично выводить ошибку если версия php старее чем 7.3.0

@MrKrasnov MrKrasnov requested a review from WerySkok May 24, 2026 06:24
@MrKrasnov
Copy link
Copy Markdown
Author

Единственно я бы обозначил в сообщении что рекомендованная версия 8.2
image

@WerySkok
Copy link
Copy Markdown
Member

@MrKrasnov Ну так апни до 8.2

@veselcraft
Copy link
Copy Markdown
Member

Единственно я бы обозначил в сообщении что рекомендованная версия 8.2 image

мне кажется, что овк уже не работает на 7.4) Учитывая что он EOL. Так что минимум ему 100% PHP 8.2

Copy link
Copy Markdown
Member

@WerySkok WerySkok left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

В самом условии тоже исправь до "8.2.0", без этого не приму

@MrKrasnov
Copy link
Copy Markdown
Author

Окей, подправил условие. 1ce4aa6

Мое мнение я бы вообще в этом ПР не трогал bootstrap.php, но дело конечно ваше) Изначально цель этого ПР была устранить ошибки при сборке компосера, а собирать композер появилась необходимость так как варнинги и ошибки светились в phpstorm. Изначально весь проект разворачиваю в докере и там vendor сам создается с необходимыми пакетами, по этому все эти танцы только ради phpstorm.

@MrKrasnov MrKrasnov requested a review from WerySkok May 25, 2026 05:40
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.

3 participants