Skip to content

Conversation

@herbertroth
Copy link
Member

No description provided.

@sonarqubecloud
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
"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
@alexz707 alexz707 added this to the 2.0.0 milestone Jan 27, 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