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

Set php 8.3 as min #810

Merged
merged 11 commits into from
Jan 15, 2025
Merged

Set php 8.3 as min #810

merged 11 commits into from
Jan 15, 2025

Conversation

herbertroth
Copy link
Member

No description provided.

Copy link

sonarqubecloud bot commented Jan 15, 2025

Quality Gate Passed Quality Gate passed

Issues
0 New issues
12 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
7.4% Duplication on New Code

See analysis details on SonarQube Cloud

@herbertroth herbertroth requested a review from alexz707 January 15, 2025 13:23
@@ -12,20 +12,20 @@
"prefer-stable": true,
"minimum-stability": "dev",
"require": {
"php": "~8.1.0 || ~8.2.0 || ~8.3.0",
"php": "~8.3.0 || ~8.4.0",
Copy link
Contributor

Choose a reason for hiding this comment

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

according our meeting in the morning this line should be removed completely right?

Copy link
Contributor

Choose a reason for hiding this comment

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

Please see: pimcore/generic-data-index-bundle#276 (comment)

It is better to always explicitly define the dependencies of a package instead of relying on the definition of another package. This is especially true for the PHP version, as you can no longer change it once a package version has been released.

Copy link
Member Author

@herbertroth herbertroth Jan 15, 2025

Choose a reason for hiding this comment

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

@alexz707 Yes, according to the meeting this should be removed. I figure out the that most of the test depends on this(see workflows). So we can refactor the test or keep this. We will talk tomorrow in the meeting :-)

@herbertroth herbertroth merged commit 2013cc6 into 2.x Jan 15, 2025
10 checks passed
@herbertroth herbertroth deleted the Set-php-8.3-as-min branch January 15, 2025 13:59
@github-actions github-actions bot locked and limited conversation to collaborators Jan 15, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants